[PR #34] [MERGED] improve handoff parsing, chaining, and banner quality #34

New issue

Closed

opened 2026-03-04 01:39:21 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 01:39:21 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/yigitkonur/cli-continues/pull/34
Author: @yigitkonur
Created: 3/3/2026
Status: ✅ Merged
Merged: 3/3/2026
Merged by: @yigitkonur

Base: main ← Head: feat/banner-weighted-presets-tab-cycle

📝 Commits (7)

ea64467 feat(cli): add weighted banner tips and tab cycling
0c2a5c0 fix: address PR review issues in banner and handoff defaults
6ac7902 fix(handoff): consume approval-mode passthrough for droid/amp/opencode
468f99f improve claude handoff parsing and chain behavior
d1ecc84 fix: make banner ctrl-c cancel cleanly without prompt race
cf85f73 keep sigint exit code consistent with ctrl-c semantics
560c8dc merge: resolve main conflicts for PR #34

📊 Changes

17 files changed (+971 additions, -80 deletions)

View changed files

➕ src/__tests__/claude-task-reconciliation.test.ts (+144 -0)
📝 src/__tests__/forward-flags.test.ts (+12 -13)
📝 src/__tests__/shared-utils.test.ts (+20 -1)
📝 src/__tests__/unit-conversions.test.ts (+29 -0)
📝 src/cli.ts (+89 -23)
📝 src/commands/dump.ts (+16 -1)
📝 src/commands/inspect.ts (+14 -1)
📝 src/commands/pick.ts (+23 -5)
📝 src/commands/resume-cmd.ts (+5 -4)
📝 src/config/verbosity.ts (+15 -0)
📝 src/display/banner.ts (+276 -15)
📝 src/parsers/claude.ts (+274 -11)
📝 src/types/tool-names.ts (+0 -2)
📝 src/utils/index.ts (+3 -2)
📝 src/utils/markdown.ts (+2 -0)
📝 src/utils/resume.ts (+40 -2)
📝 src/utils/tool-extraction.ts (+9 -0)

📄 Description

what changed

improved claude handoff parsing so important events are no longer dropped (including exitplanmode and richer task status reconciliation)
fixed file tracking for mcp__morph__edit_file so modified-file reporting is complete
improved subagent result rendering and pending-task filtering to remove false positives
added compacted-session chaining by default, with --no-chain as an explicit opt-out
threaded preset/config behavior through resume, pick, inspect, and dump flows
added and updated tests for reconciliation, extraction, and rendering behavior

why

the goal was to make claude -> codex handoff context much more reliable and higher-signal by fixing missing event coverage, reducing misleading status output, and making verbosity/chaining behavior predictable.

verification

pnpm vitest (9 files, 715 tests passed)
pnpm build
node dist/cli.js inspect 1027312a-9493-49c8-83ba-a925afbf3ab4 --write-md /tmp/codex-output-chain-on.md
node dist/cli.js --no-chain inspect 1027312a-9493-49c8-83ba-a925afbf3ab4 --write-md /tmp/codex-output-chain-off.md
verified expected inspection signals:
- tool calls: 633
- files modified: 38
- includes src/tools/send-message.ts
- no false pending watch ci run until completion
- no false ⚠️ completed
- chain context present by default and absent with --no-chain

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/yigitkonur/cli-continues/pull/34 **Author:** [@yigitkonur](https://github.com/yigitkonur) **Created:** 3/3/2026 **Status:** ✅ Merged **Merged:** 3/3/2026 **Merged by:** [@yigitkonur](https://github.com/yigitkonur) **Base:** `main` ← **Head:** `feat/banner-weighted-presets-tab-cycle` --- ### 📝 Commits (7) - [`ea64467`](https://github.com/yigitkonur/cli-continues/commit/ea64467af8eaa8f8660eee76a14149e0905e68e2) feat(cli): add weighted banner tips and tab cycling - [`0c2a5c0`](https://github.com/yigitkonur/cli-continues/commit/0c2a5c02e3efdb49758e939fce7a5303a0851598) fix: address PR review issues in banner and handoff defaults - [`6ac7902`](https://github.com/yigitkonur/cli-continues/commit/6ac790202ff5a3c5a82598842e3db784792e414b) fix(handoff): consume approval-mode passthrough for droid/amp/opencode - [`468f99f`](https://github.com/yigitkonur/cli-continues/commit/468f99f4256754bada84641ad397ed793f3f11d6) improve claude handoff parsing and chain behavior - [`d1ecc84`](https://github.com/yigitkonur/cli-continues/commit/d1ecc843c411895e571b3dfa2b8d3b61f3bf7284) fix: make banner ctrl-c cancel cleanly without prompt race - [`cf85f73`](https://github.com/yigitkonur/cli-continues/commit/cf85f739440315f9f1c94ab533a9fb72c03937de) keep sigint exit code consistent with ctrl-c semantics - [`560c8dc`](https://github.com/yigitkonur/cli-continues/commit/560c8dcb56cff4c174775737df53e85a5c232a01) merge: resolve main conflicts for PR #34 ### 📊 Changes **17 files changed** (+971 additions, -80 deletions) <details> <summary>View changed files</summary> ➕ `src/__tests__/claude-task-reconciliation.test.ts` (+144 -0) 📝 `src/__tests__/forward-flags.test.ts` (+12 -13) 📝 `src/__tests__/shared-utils.test.ts` (+20 -1) 📝 `src/__tests__/unit-conversions.test.ts` (+29 -0) 📝 `src/cli.ts` (+89 -23) 📝 `src/commands/dump.ts` (+16 -1) 📝 `src/commands/inspect.ts` (+14 -1) 📝 `src/commands/pick.ts` (+23 -5) 📝 `src/commands/resume-cmd.ts` (+5 -4) 📝 `src/config/verbosity.ts` (+15 -0) 📝 `src/display/banner.ts` (+276 -15) 📝 `src/parsers/claude.ts` (+274 -11) 📝 `src/types/tool-names.ts` (+0 -2) 📝 `src/utils/index.ts` (+3 -2) 📝 `src/utils/markdown.ts` (+2 -0) 📝 `src/utils/resume.ts` (+40 -2) 📝 `src/utils/tool-extraction.ts` (+9 -0) </details> ### 📄 Description ## what changed - improved claude handoff parsing so important events are no longer dropped (including exitplanmode and richer task status reconciliation) - fixed file tracking for `mcp__morph__edit_file` so modified-file reporting is complete - improved subagent result rendering and pending-task filtering to remove false positives - added compacted-session chaining by default, with `--no-chain` as an explicit opt-out - threaded preset/config behavior through resume, pick, inspect, and dump flows - added and updated tests for reconciliation, extraction, and rendering behavior ## why the goal was to make claude -> codex handoff context much more reliable and higher-signal by fixing missing event coverage, reducing misleading status output, and making verbosity/chaining behavior predictable. ## verification - `pnpm vitest` (9 files, 715 tests passed) - `pnpm build` - `node dist/cli.js inspect 1027312a-9493-49c8-83ba-a925afbf3ab4 --write-md /tmp/codex-output-chain-on.md` - `node dist/cli.js --no-chain inspect 1027312a-9493-49c8-83ba-a925afbf3ab4 --write-md /tmp/codex-output-chain-off.md` - verified expected inspection signals: - tool calls: `633` - files modified: `38` - includes `src/tools/send-message.ts` - no false pending `watch ci run until completion` - no false `⚠️ completed` - chain context present by default and absent with `--no-chain` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-04 01:39:21 +03:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/cli-continues#34

No description provided.

Rows
Columns