Skip to content

fix(codex): once-per-turn outcome review through the Stop hook (main) - #59

Open
e1daru wants to merge 14 commits into
mainfrom
feat/codex-24-review-main
Open

fix(codex): once-per-turn outcome review through the Stop hook (main)#59
e1daru wants to merge 14 commits into
mainfrom
feat/codex-24-review-main

Conversation

@e1daru

@e1daru e1daru commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Published counterpart of #42, already merged into pre-main. Installed plugins run what is on main, so this is how that change reaches users.

The review on Codex, and Stop fixes shared by both hosts:

  • any later Stop of a reviewed turn is its follow-up, so the review never repeats and one outcome is sent;
  • use is measured on the first answer only, since the review lists the lesson titles;
  • a reply to a message typed after the review is stored as its own item.

main carries no sources or tests. Every file here matches pre-main at the merge of #42, with two differences: bundles lose their inline sourcemap line, and paths that don't ship (sources, tests, labs/, verify.yml) are left out. Applied to pre-main before this series, that rule reproduces today's main byte for byte.

Merge order

14 of 16. Merge after #58. This PR targets main directly, so until #58 merges its diff also shows the earlier PRs in the series.

Test plan

  • Sources: fix(codex): once-per-turn outcome review through the Stop hook #42's CI on pre-main. At the pre-main tip: Claude Code 2341 on source and dist, Codex 635 on both, manifests, bundles match their source.
  • This tip: every shipped script passes node --check, and none carries an inline sourcemap.
  • Both hosts, offline in a throwaway home: session-start and capture --stop exit 0 with valid output, and both MCP servers answer initialize and tools/list (mubit-memory@0.13.2, 7 tools).
  • leakcheck --strict exit 0.

🤖 Generated with Claude Code

e1daru and others added 14 commits September 29, 2026 13:33
@e1daru @claude
ci(leakcheck): scan every commit a push publishes, and read dotfiles …
c64b5ca 
…(main)

Published form of #26: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
feat(scorecard): session log, memory handles, settings and the card r…
f892d58 
…enderer (main)

Published form of #27: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
feat(scorecard): correction signal, tool intent and per-entry outcome…
8df438a 
… credit (main)

Published form of #28: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
feat(outcome): credit memory by its short id; the once-per-turn revie…
2f27570 
…w text (main)

Published form of #29: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
feat(scorecard): handles on memory lines and per-entry recall data (m…
87468b4 
…ain)

Published form of #30: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
feat(scorecard): measure each lesson, ask for the review, print the c…
0375ab3 
…ard (main)

Published form of #31: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
chore: drop internal spec section numbers from source comments (main)
0b75dfe 
Published form of #32: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
docs: skills, docs and manifests describe what the plugin does (main)
02cc42c 
Published form of #33: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
chore: comments and messages describe what the plugin does (main)
6698c81 
Published form of #34: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
docs: placeholder ids in the README screenshots (main)
97767e4 
Published form of #36: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
fix(codex): Stop-hook feedback in a rollout is not the user's words (…
2fe3b1e 
…main)

Published form of #38: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
feat(codex): explain memory ids in the session-start block (main)
11b378b 
Published form of #39: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
fix(codex): record mubit_outcome and mubit_learned calls in the sessi…
f352b1b 
…on log (main)

Published form of #41: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@e1daru @claude
fix(codex): once-per-turn outcome review through the Stop hook (main)
d92510d 
Published form of #42: the rebuilt bundles without their inline
sourcemap line, and the docs, skills and manifests that ship. Sources
and tests stay on pre-main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings September 29, 2026 12:41
@e1daru e1daru mentioned this pull request Sep 29, 2026
Open
4 tasks done
Copilot AI reviewed Sep 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@e1daru