Skip to content

Beta: usable setup (combines #139 + #140, fixes 5 dogfood blockers) - #142

Closed
PatrickSys wants to merge 1 commit into
masterfrom
cursor/beta-usable-setup-d85c
Closed

PatrickSys wants to merge 1 commit into
masterfrom
cursor/beta-usable-setup-d85c

Conversation

@PatrickSys

Copy link
Copy Markdown
Owner

Draft. Do not merge or publish yet.

init now does the multi-tool setup from #139 and saves the launch command the way #140 does. It detects Claude Code, Codex, Cursor and OpenCode and writes a config for each one you pick.

Fixes:

  1. Docs match the new init, and they note it isn't in npm 2.2.0 yet.
  2. map builds the index on a fresh project, so it no longer reports 0 files.
  3. Adds --version / -v.
  4. Adds a Codex setup line (codex mcp add codebase-context -- npx -y codebase-context).
  5. A weak search result now explains itself and suggests a narrower next step.

Tests: type-check passes and 679/679 tests pass.

Still open: re-running the 5 dogfood questions, a Windows install check, memory use on a mid-size repo, and search quality. #139 and #140 are left as they are.

Tracked in IdeaSpine: https://github.com/PatrickSys/ideaspine/issues/17

Init configures Claude Code, Codex, Cursor, and OpenCode together and saves the Node binary plus the package file from the npx cache. map indexes before it prints, --version prints the package version, and a low-confidence search tells the agent a weak result is not proof the code is missing.

Copy link
Copy Markdown
Owner Author

@PatrickSys

Copy link
Copy Markdown
Owner Author

Closing this superseded draft because its commit is included in #131's public branch, alongside the publication and source-validity fixes. The installed candidate recorded on #131 exercised the Windows install and five original Express questions: frozen retrieval found an expected file in the top five for 14/21 queries, with Hono peak sampled server RSS of 1.516 GB and an explicit graph limitation. PR131 remains a draft while its final repair is validated; nothing is being claimed merged or released. PR140's extra client targets and HTTP setup remain separate. This branch is being retained.

@PatrickSys PatrickSys closed this Oct 30, 2026
@cursor

cursor Bot commented Oct 30, 2026

Copy link
Copy Markdown

Adversarial review (maintainer)

Verdict: fix first — do not merge until rebased onto current master.

What I verified: Checked out the branch, pnpm build, 680/680 tests (after fix), MCP initialize OK, CLI reindex/map/search on this repo and sindresorhus/is; map auto-index on fresh tree works.

What I fixed (pushed 0265d4d): The branch still used argv[1] endsWith('index.js') entry detection, so npm-style bin symlinks silently no-op (npx codebase-context --version prints nothing). Ported canonical realpath entrypoint matching from master and added a symlink --version regression test.

Blockers left:

  1. Stale base — branch predates master (e87020f, 5db8705); would regress model-free indexing / Hono / smol-toml init unless rebased.
  2. Hard merge conflict with First-run fixes: OPENAI_BASE_URL and CLI entrypoint regression test #145 on src/index.ts, src/cli-init.ts, docs, and several tests — rebase Beta: usable setup (combines #139 + #140, fixes 5 dogfood blockers) #142 after First-run fixes: OPENAI_BASE_URL and CLI entrypoint regression test #145 or resolve manually.
  3. Missing First-run fixes: OPENAI_BASE_URL and CLI entrypoint regression test #145 — no OPENAI_BASE_URL wiring here.
  4. Dogfood nit: CLI search can show confidence >1.0 (score boost not capped in display); unrelated to this PR but confusing for agents.

Suggested order: Merge #145 first, rebase #142, rerun full suite + init wizard smoke.

@cursor

cursor Bot commented Oct 30, 2026

Copy link
Copy Markdown

Review update (post–master merge)

Verdict: merge after #145 (was fix-first; now current with master).

Done: Merged origin/master into this branch (cc18718, normal merge, no rebase/force-push). Conflicts resolved by keeping master’s init preflight/smol-toml, index-artifact map path, and embedding CLI flags, plus this branch’s canonical symlink entrypoint + tests/cli-version.test.ts symlink case and search-quality hint text/docs.

Verified: pnpm build + 1304 passed (7 skipped). Local merge of cursor/first-run-fixes-16ec on top applied cleanly (no conflict markers); combined embedding tests pass; test merge was not pushed to this branch.

Still: Land #145 first, then merge this PR. npm publish remains out of scope.

cursor Bot pushed a commit that referenced this pull request Oct 30, 2026
Bring PR #142 up to date with model-free indexing, init preflight, and
master CLI entrypoint detection while keeping symlink --version test and
search-quality hints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants