Skip to content

fix(ship-check): pass Codex attribution model - #29

Merged
aliasunder merged 3 commits into
mainfrom
fix/codex-model-attribution
Sep 21, 2026
Merged

aliasunder merged 3 commits into
mainfrom
fix/codex-model-attribution

Conversation

@aliasunder

Copy link
Copy Markdown
Owner

Summary

  • pass the exact Codex model ID from the ship-check dispatcher to every committing phase agent
  • keep Claude and OpenCode attribution tied to their runtime-exposed model identifiers
  • preserve per-run model overrides without adding model pins to agent definitions or generated files
  • carry the resolved attribution ID into inline pr-monitor runs

Verification

  • plan-check: ready with 0 findings
  • Codex sync generator: 17 tests passed
  • disposable Codex sync: 6 agents generated; second check reported 0 changes
  • generated-agent contract check passed across all four committing reviewers
  • git diff --check passed

Comment thread plugins/ship-check/agents/bug-checker.md
Comment thread plugins/ship-check/skills/ship-check/SKILL.md Outdated
@umm-actually

umm-actually Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

umm-actually re-reviewed at a525756

No new findings (3 tracked finding(s) across all runs).

Context notes
  • Priority docs already in context: plugins/ship-check/agents/bug-checker.md, plugins/ship-check/agents/code-quality-reviewer.md, plugins/ship-check/agents/pr-reviewer.md, plugins/ship-check/agents/test-auditor.md, plugins/ship-check/skills/ship-check/SKILL.md, plugins/ship-check/skills/pr-monitor/SKILL.md

umm-actually · deepseek/deepseek-v4-flash-0731

Comment thread plugins/ship-check/skills/ship-check/SKILL.md
@aliasunder
aliasunder merged commit a525756 into main Sep 21, 2026
7 checks passed
@aliasunder
aliasunder deleted the fix/codex-model-attribution branch September 21, 2026 22:56
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.

1 participant