Skip to content

fix: refactor: one module owns the /api/v1 paths used by ai-entry, index-build and the manifest - #659

Merged
mvilacad merged 2 commits into
mainfrom
fix/652-refactor-one-module-owns-the-api
Oct 26, 2026
Merged

mvilacad merged 2 commits into
mainfrom
fix/652-refactor-one-module-owns-the-api

Conversation

@mvilacad

Copy link
Copy Markdown
Contributor

Closes #652

Regression test (written first)

  • Test reproduces the problem and fails on main (paste the failing output)
  • Same test passes with the fix

Change

Checklist

  • bun run validate passes
  • Changeset (bun changeset) if the published package changes
  • Docs updated if behavior or API changed

…aths

SITE was declared in 9 places and ai-entry, index-build, the manifest writers, the MCP server and endpoints.json each spelled api/v1 URLs by hand. API, API_DIR, API_PATHS and apiUrl live in src/cli/agent-guide.ts; manifest --check reports no drift. Fail before: the new api-paths test fails (missing exports, then 1 hand-spelled path); pass after: 115 related tests. Closes #652
@mvilacad
mvilacad force-pushed the fix/652-refactor-one-module-owns-the-api branch from 312651f to ed7fc29 Compare October 26, 2026 00:58
@mvilacad
mvilacad marked this pull request as ready for review October 26, 2026 00:58
@mvilacad
mvilacad merged commit 42a8947 into main Oct 26, 2026
@mvilacad
mvilacad deleted the fix/652-refactor-one-module-owns-the-api branch October 26, 2026 00:59
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.

refactor: one module owns the /api/v1 paths used by ai-entry, index-build and the manifest

1 participant