docs(mcp): MCP gap analysis — IHalClient pivot, SwapTools tuple, and design fixes#84

Merged

brenpike merged 29 commits into

mainfrom

docs/mcp-gap-analysis

May 1, 2026

Commits on Apr 29, 2026

docs: update mcp docs for IHalClient pivot, scoped HalToolCollectionManager, and gap resolutions

brenpike
and
claude
committed
docs: fix remaining stale GetHalAsync and HttpClient references in mcp docs

brenpike
and
claude
committed
docs: fix Codex review findings in mcp requirements and architecture

brenpike
and
claude
committed
docs: fix case normalization order in tool-name sanitization algorithm

brenpike
and
claude
committed
docs: fix relative URI handling and NavigateToRootTool DI contradiction in mcp docs

brenpike
and
claude
committed
docs: enforce ScopeRequests=true in WithHalApi registration (REQ-46)

brenpike
and
claude
committed
docs: fix IHalClient DI registration in Integration Story example

brenpike
and
claude
committed
docs: add query-strip, single-session constraint, and empty-LinkObjects guard to mcp docs

brenpike
and
claude
committed
docs: add HalClientResult envelope and GetResultAsync to client/MCP docs

brenpike
and
claude
committed
docs: add counter-based tool name collision dedup to mcp requirements and architecture

brenpike
and
claude
committed
update agent framework
brenpike
committed
update agent framework
brenpike
committed
docs: fix HalClientResult type ambiguity and enforce href sanitization in log call sites

brenpike
and
claude
committed
docs: chain .AddHalOptions() to IHalClient registration in MCP integration story

brenpike
and
claude
committed
Merge remote-tracking branch 'origin/main' into docs/mcp-gap-analysis
brenpike
committed
docs: fix AddHalOptions missing configure delegate in MCP integration example

brenpike
and
claude
committed
docs: anchor REQ-05 scheme-strip to string start to protect relative hrefs with URL query params

brenpike
and
claude
committed
docs: add pageInfo to comments connections in github-pr-review-graphql.md

brenpike
and
claude
committed
docs: add pageInfo to reviewThreads connections in github-pr-review-graphql.md

brenpike
and
claude
committed
docs: add after cursor args to GraphQL queries; add review summary fetch to run-codex-review-loop step 2

brenpike
and
claude
committed
docs: add per-thread comment pagination query to github-pr-review-graphql.md

brenpike
and
claude
committed
docs: add review summary fetch to watch-pr-feedback detection path

brenpike
and
claude
committed
docs: add Fetch Reviews query to github-pr-review-graphql.md

brenpike
and
claude
committed
docs: make REQ-05 scheme-strip regex case-insensitive

brenpike
and
claude
committed
docs: fix per-thread comment pagination cursor wording in github-pr-review-graphql.md

brenpike
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs(mcp): MCP gap analysis — IHalClient pivot, SwapTools tuple, and design fixes#84

docs(mcp): MCP gap analysis — IHalClient pivot, SwapTools tuple, and design fixes#84
brenpike merged 29 commits into
mainfrom
docs/mcp-gap-analysis

Commits on Apr 29, 2026

docs: update mcp docs for IHalClient pivot, scoped HalToolCollectionManager, and gap resolutions

docs: fix remaining stale GetHalAsync and HttpClient references in mcp docs

docs: fix Codex review findings in mcp requirements and architecture

docs: fix case normalization order in tool-name sanitization algorithm

docs: fix relative URI handling and NavigateToRootTool DI contradiction in mcp docs

docs: enforce ScopeRequests=true in WithHalApi registration (REQ-46)

docs: fix IHalClient DI registration in Integration Story example

docs: add query-strip, single-session constraint, and empty-LinkObjects guard to mcp docs

docs: add HalClientResult envelope and GetResultAsync to client/MCP docs

docs: add counter-based tool name collision dedup to mcp requirements and architecture

update agent framework

update agent framework

docs: fix HalClientResult type ambiguity and enforce href sanitization in log call sites

docs: chain .AddHalOptions() to IHalClient registration in MCP integration story

Merge remote-tracking branch 'origin/main' into docs/mcp-gap-analysis

docs: fix AddHalOptions missing configure delegate in MCP integration example

docs: anchor REQ-05 scheme-strip to string start to protect relative hrefs with URL query params

docs: add pageInfo to comments connections in github-pr-review-graphql.md

docs: add pageInfo to reviewThreads connections in github-pr-review-graphql.md

docs: add after cursor args to GraphQL queries; add review summary fetch to run-codex-review-loop step 2

docs: add per-thread comment pagination query to github-pr-review-graphql.md

docs: add review summary fetch to watch-pr-feedback detection path

docs: add Fetch Reviews query to github-pr-review-graphql.md

docs: make REQ-05 scheme-strip regex case-insensitive

docs: fix per-thread comment pagination cursor wording in github-pr-review-graphql.md

Commits on May 1, 2026

docs(mcp): add SwapTools tuple return and NavigateToRootTool InvokeAsync flow

docs(mcp): fix McpServerOptions resolution and removedCount calculation

Merge remote-tracking branch 'origin/main' into docs/mcp-gap-analysis

docs(mcp): fix addedCount hardcoded subtraction in SwapTools algorithm