refactor: modernize CLI infrastructure and UX#20
Merged
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…K ranges Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tive fetch - Remove axios, replace all HTTP calls with native fetch + AbortController - Upgrade marked 11→15, typescript 5.3→5.9, open 10→11, @clack/prompts 1.2 - Upgrade @types/node 22, tsx 4.21 - Fix marked.setOptions→marked.use API for marked 15 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace shell-based postbuild with Node.js copy-assets script for cross-platform compatibility. Add test step to CI, remove redundant tsc --noEmit check. Fix test to use XDG config path. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix --today filter: compare Date objects instead of formatted strings to handle cross-year events correctly - Convert remaining .replace() calls in docs.ts to fmt() for consistency - Rename evictOldest to evictStalest to accurately reflect FIFO behavior - Improve abort timeout error messages: show "Request timed out" instead of cryptic "The operation was aborted" across all fetch call sites - Restore build step in npm test for local dev safety Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #19
Summary
axioswith nativefetch+AbortControlleracross calendar, status, and docs modules~/.nbtca/legacy fallbackfmt()helper, vim-keys context toggle, iterative directory browsing, LRU cache evictionCommits (9)
refactor: XDG config directory with legacy fallbackrefactor: add i18n fmt helper, replace .replace() chainsfix: vim-keys text input conflict, shared SIGINT handler, extended CJK rangesfix: remove dead code in links.ts, fix any type in status.tsfix: show year in calendar dates for cross-year eventsrefactor: convert recursive directory browsing to iterative loopsfeat: LRU cache eviction and content-aware doc searchchore: upgrade deps (marked 15, TS 5.9, open 11), remove axios for native fetchfeat: cross-platform build script and CI test stepTest plan
npm run build)tsc --noEmit)npm test)🤖 Generated with Claude Code