fix(cli): rerender splash banner with clean figlet output + README np…#416
Closed
rohitg00 wants to merge 4 commits into
Closed
fix(cli): rerender splash banner with clean figlet output + README np…#416rohitg00 wants to merge 4 commits into
rohitg00 wants to merge 4 commits into
Conversation
…x caveat
Two small fixes from local v0.9.15 testing:
1. The hand-drawn ASCII in `src/cli/splash.ts` was rendering "agentmemory"
with broken middle glyphs — `_ _` instead of `_ __` around the 'n'
and merged 'tm/me' segments. Replaced with verified `figlet
agentmemory` standard-font output (70 cols, 6 rows, regenerable).
Compact + minimal banners untouched.
2. Added a "Recommended: install globally" section to README's Quick
Start. Explains the npx per-version cache gotcha (running
`npx ...@0.9.14` then `npx @agentmemory/agentmemory` may serve the
stale cached version) and the three workarounds:
- `npm install -g @agentmemory/agentmemory` (recommended)
- `npx -y @agentmemory/agentmemory@latest` (forces fresh fetch)
- `rm -rf ~/.npm/_npx && npx ...` (one-shot cache clear)
Notes that v0.9.16 ships an interactive global-install prompt
inline on first npx run.
Build clean (44.46 kB cli.mjs). All 954 tests pass.
… card README: install was buried at line 306 below benchmarks, comparison tables, and the agents grid. Moved a 4-line install block right under the nav anchors at the top, with the bare `agentmemory` command from the global install path leading. Kept the deeper Quick Start section below — this is the "front door" version. website (Install.tsx): the three-card install flow now leads with `npm install -g @agentmemory/agentmemory` (step 1), then bare `agentmemory` + `agentmemory demo`. Section title changed from "THREE COMMANDS. ANY AGENT." to "ONE INSTALL. ANY AGENT." and the lede notes the npx fallback for users who want zero-install. website/lib/generated-meta.json refreshed by the build (auto-derived from package.json + walked test count; pre-existing feedback rule). Build clean (website + main both pass).
README: annotate `rm -rf ~/.npm/_npx` as POSIX-only, point Windows users at `Remove-Item ...\npm-cache\_npx`. Mark `npx -y @...@latest` as the cross-platform option. Applies to both install-section and quick-start blocks (lines 87, 363). website Install.tsx: hint on step 1 now says "STEPS 2/3 NEED THIS" so users who skip can't get stuck on `agentmemory` not found. Added a fourth card "PREFER ZERO-INSTALL? USE NPX" with the bare npx command and a pointer to the README caveat — covers the skip path without making step 1 ambiguous. Skipped (verified-false): coderabbit flagged the splash banner's "70 columns wide" comment as wrong (claimed rows 3+5 are 76/77). They counted source-string chars including the `\\` escapes; runtime is 70 on all 6 rows (node length check), which is what alignment cares about. Comment is accurate, no change needed.
Drift accumulated across the website from v0.9.0 through v0.9.15. Each section caught up: CommandCenter: - "iii CONSOLE · OPTIONAL" → "iii CONSOLE · FIRST-CLASS". User pushback: the console gives engine-level visibility (workers, functions, queues, traces) and is now installed inline by the agentmemory CLI on first run from v0.9.16. Not optional. - Bullet "49 HTTP TRIGGERS" → "107 HTTP ENDPOINTS" to match generated-meta.json restEndpoints count. Dropped the stale "33 REGISTERED FUNCTIONS" subhead. - Section lede now mentions both UIs are first-class + inline- installed. Compare: - MCP TOOLS our column 44 → 51 (matches generated-meta.json). - New REST ENDPOINTS row (107) — competitors don't ship a REST shape per their docs, so dashes for mem0/letta/cognee. - New NATIVE PLUGINS row listing the 6 first-party agents + Cursor/Gemini CLI. Agents: - FEATURED expanded from 4 to 6 (added pi + OpenHuman). Codex CLI pitch updated to "6 hooks + MCP · native plugin" reflecting the hooks addition in PR #383. - Section title "FOUR FIRST-PARTY" → "SIX FIRST-PARTY". - Section lede now mentions `agentmemory connect <agent>` (shipped in v0.9.15 PR #402). Hero: - "START IN 60 SECONDS" CTA → "START IN 30 SECONDS". Cold install + engine spawn measured 8-12s on v0.9.15 with the native binary path; 60s was the v0.9.0-era Docker-first claim. generated-meta.json regenerated by the build (auto-derive rule). Website build clean.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThis PR updates documentation, website marketing content, and metadata to reflect a new product version. Installation instructions are consolidated in the README and website component, the featured agents list is expanded with three new entries, capability comparisons are refreshed, console/command center messaging is updated, and CLI splash art and build metadata are bumped. ChangesInstallation & Platform Marketing Content
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
…x caveat
Two small fixes from local v0.9.15 testing:
The hand-drawn ASCII in
src/cli/splash.tswas rendering "agentmemory"with broken middle glyphs —
_ _instead of_ __around the 'n'and merged 'tm/me' segments. Replaced with verified
figlet agentmemorystandard-font output (70 cols, 6 rows, regenerable).Compact + minimal banners untouched.
Added a "Recommended: install globally" section to README's Quick
Start. Explains the npx per-version cache gotcha (running
npx ...@0.9.14thennpx @agentmemory/agentmemorymay serve thestale cached version) and the three workarounds:
npm install -g @agentmemory/agentmemory(recommended)npx -y @agentmemory/agentmemory@latest(forces fresh fetch)rm -rf ~/.npm/_npx && npx ...(one-shot cache clear)Notes that v0.9.16 ships an interactive global-install prompt
inline on first npx run.
Build clean (44.46 kB cli.mjs). All 954 tests pass.
Summary by CodeRabbit
Documentation
New Features
Updates