Skip to content

fix: unify the local agent default port to 3002 - #208

Merged
M0nsxx merged 1 commit into
mainfrom
fix/port-unification-3002
Oct 25, 2026
Merged

M0nsxx merged 1 commit into
mainfrom
fix/port-unification-3002

Conversation

@Eras256

@Eras256 Eras256 commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • PR fix(cli): status default port didn't match the documented local quickstart #205 fixed the CLI's own --url default (3001 -> 3002). This repo had 6 more files hardcoding the same stale default: both SDK clients (TS/Python), the MCP server's AGENT_API_URL default + its own usage example, and both example files in examples/audit-forensic-bridge.
  • Left alone on purpose: packages/cli/CHANGELOG.md's entry documents a past release's behavior (historical record), and actions/verify-audit-cid/dist/index.js's "3001" substring is a hex constant (0xbc423001), not the port - confirmed before touching it.

Test plan

  • SDK still builds (tsc) and all 56 jest tests pass after the change

🤖 Generated with Claude Code

…on, and examples

PR #205 (today) fixed the CLI's own --url default. This repo has 6
more files hardcoding the same stale 3001 default - TypeScript and
Python SDK clients, the MCP server's AGENT_API_URL default and its own
usage example, and both example files in audit-forensic-bridge.

Left two things alone on purpose: packages/cli/CHANGELOG.md's own
entry documents a past release's behavior (historical record, not a
current-state claim), and actions/verify-audit-cid/dist/index.js's
"3001" substring is a hex constant (0xbc423001), not the port number -
confirmed before touching it, not assumed from the grep match.

Verified: SDK still builds and all 56 jest tests pass after the change.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@M0nsxx M0nsxx left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI green, same port fix on the SDK source side, 6 more files (item 4). Ready to merge.

@M0nsxx
M0nsxx merged commit 64a341d into main Oct 25, 2026
15 checks passed
@Eras256
Eras256 deleted the fix/port-unification-3002 branch November 4, 2026 01:16
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