Skip to content

feat(codex): scorecard and review on by default, auto-approve the two credit tools (main) - #61

Open
e1daru wants to merge 16 commits into
mainfrom
feat/codex-25-defaults-main
Open

feat(codex): scorecard and review on by default, auto-approve the two credit tools (main)#61
e1daru wants to merge 16 commits into
mainfrom
feat/codex-25-defaults-main

Conversation

@e1daru

@e1daru e1daru commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Summary

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

This turns the scorecard and the review on for Codex.

  • sessionScore: full and outcomeReview: stop on both hosts. On Codex the card shows under the reply in the interactive UI; codex exec does not print it.
  • scripts/setup.mjs auto-approves only mubit_outcome and mubit_learned, so the review doesn't raise an approval prompt each turn. Values the user set, other server settings and env entries are kept.
  • The Codex MCP server knows it runs under Codex, and the sign-in messages use the Codex skill spelling.

main carries no sources or tests. Every file here matches pre-main at the merge of #44, 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

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

After this one merges, main is pre-main published. #35, #37, #40 and #45 changed only tests and CI, so they have no counterpart.

Test plan

  • Sources: feat(codex): scorecard and review on by default, auto-approve the two credit tools #44'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.
  • Manual: a fresh install on each host shows the card under a reply that used a lesson. The version is still 0.13.2, so existing installs keep their cached copy until the version is bumped or the cache is cleared.

🤖 Generated with Claude Code

e1daru and others added 16 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>
@e1daru @claude
test(codex): session scorecard end to end on Codex payloads (main)
cabf93e 
Published form of #43: 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): scorecard and review on by default, auto-approve the two…
7d1a3c1 
… credit tools (main)

Published form of #44: 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
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.

@e1daru e1daru mentioned this pull request Sep 29, 2026
Open
4 tasks done
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