Skip to content

fix: harden Windows CLI discovery and summary detection - #15

Open
mradwankhalil wants to merge 2 commits into
openchamber:mainfrom
mradwankhalil:fix/windows-cli-resolution-and-summary-envelope
Open

fix: harden Windows CLI discovery and summary detection#15
mradwankhalil wants to merge 2 commits into
openchamber:mainfrom
mradwankhalil:fix/windows-cli-resolution-and-summary-envelope

Conversation

@mradwankhalil

@mradwankhalil mradwankhalil commented Sep 5, 2026

Copy link
Copy Markdown

Summary

  • Probe Windows Claude CLI wrapper variants (.cmd, .exe, .bat) and use platform-safe path joins.
  • Convert file URLs to filesystem paths in smoke subprocesses.
  • Recognize OpenCode update-summary envelopes when prose appears between the prior-summary paragraph and template instructions.
  • Preserve ordinary prior-summary mentions as normal requests.

Fixes #13
Fixes #14
Follow-up to #11.

Merge note

This PR intentionally touches src/request-kind.ts because it broadens the strict classifier introduced by PR #11. If #11 merges first, rebase this branch and retain the broader intervening-prose pattern; if this PR is reviewed first, apply its classifier hunk on top of #11 before merge.

Verification

  • bunx tsc -p tsconfig.json --noEmit
  • bun test/smoke.ts

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.

Recognize OpenCode update-summary envelopes with intervening instructions Windows Claude CLI discovery misses executable variants and file URL paths

1 participant