Normalize agent model footer metadata

[sepo-agent-app]

Summary

• Extend the Normalize Codex GPT-5 answer footer metadata #400 model-display helper to choose from both session-reported and requested model evidence.
• Prefer concrete Claude model IDs over provider-local aliases such as default, while keeping alias-only fallbacks visibly non-canonical.
• Preserve the compact provider | model | reasoning effort | runner footer contract and Codex GPT-5 reasoning suffix normalization.
• Update AGENT_DISPLAY_MODEL docs to describe normalized footer behavior.

Tests

• npm --prefix .agent run build
• node --test .agent/dist/__tests__/model-display.test.js
• node --test .agent/dist/__tests__/envelope.test.js
• node --test .agent/dist/__tests__/docs-validation.test.js
• git diff --check

Builds on #400.
Closes #406

[sepo-agent-app]

Sepo is dispatching follow-up automation.

Source	Next	Target	Round	Status
implement	review	PR #407	3 / 40	Dispatched

Reason: agent planner selected review: Implementation succeeded and produced open stacked PR #407 for issue #406; automated review is warranted before stopping or human merge decisions.

[sepo-agent-app]

Rubrics Review

Total Score	Verdict	Rubrics Scored
98	PASS	7

Dimension	Rubric	Result	Score	Evidence
coding_workflow / generic	Read docs and linked context first	pass	6/6	PR #407 explicitly builds on #400 and closes #406; the diff extends the existing #400 model-display helper instead of adding a parallel display path.
coding_workflow / generic	Reuse existing code	partial	7/8	run.ts continues to emit the same model output and delegates footer formatting to buildModelDisplay. Minor caveat: Codex suffix knowledge still lives separately from adapter model-selection semantics.
coding_workflow / generic	Make surgical changes	pass	8/8	The PR touches four focused files: one helper, one test file, one run.ts call site, and one config doc row. No workflow or unrelated refactor churn.
coding_style / generic	Prefer simple implementations	pass	7/7	Claude alias handling is a small explicit branch in selectDisplayModel with no new settings, routes, or abstraction layers.
coding_workflow / generic	Prefer explicit structured inputs	pass	7/7	buildModelDisplay now accepts separate reportedSessionModel and requestedModel fields, and run.ts passes both instead of collapsing them before normalization.
coding_workflow / generic	Keep docs in sync	pass	7/7	AGENT_DISPLAY_MODEL docs were updated to describe normalized footer behavior from session and requested-model evidence.
communication / generic	Keep status comments compact	pass	6/6	The implementation preserves the compact footer contract: [provider | model | reasoning effort | runner], with one markdown-ready model_display string.

Notes

• The strongest score drivers are scope fit and contract preservation: Normalize agent model footer metadata #407 handles both Normalize Codex GPT-5 answer footer metadata #400’s Codex GPT-5 reasoning suffix normalization and Clean up agent model display normalization across providers #406’s Claude/default alias case through the same helper.
• Regression coverage is focused and matches the requested cases: concrete Claude requested model over default, concrete session model preference, alias-only fallback, partial session metadata fallback, and Codex suffix normalization.
• Smallest useful follow-up: if model alias or Codex suffix rules keep growing, extract shared provider model parsing semantics so display normalization and ACPX model selection cannot drift.

Findings

• INFO: Under reuse-existing-code, the remaining duplication of Codex suffix concepts is acceptable for this focused PR, but it is the only rubric-specific maintainability caveat.

Final Rubric Verdict

PASS

---

codex | gpt-5.5/xhigh | Macmini-runner-1

[sepo-agent-app]

AI Review Synthesis

Dual-agent review by Claude and Codex.
Requested by @lolipopshock.

Summary of PR/Issue

• PR Normalize agent model footer metadata #407 closes Clean up agent model display normalization across providers #406 by normalizing agent model footer metadata across providers.
• It builds on Normalize Codex GPT-5 answer footer metadata #400’s Codex GPT-5 footer normalization instead of adding a parallel display path.
• The change passes separate session-reported and requested model evidence into the display helper, so Claude aliases like default no longer hide a concrete resolved model such as claude-opus-4-8.
• It also preserves the compact provider | model | reasoning effort | runner footer contract and updates the AGENT_DISPLAY_MODEL docs.

Review

All current reviewer artifacts converge on code readiness: Claude and Codex both conclude SHIP, and rubrics review passes. The only severity mismatch is operational: Claude labels the stacked base on open PR #400 as a warning, while the other reviewers report no blocking or warning-level code issues.

Issue	Severity	Description
No actionable branch-change issues remain	INFO	Current reviews and PR state support the requested normalization; remaining notes are optional maintainability or merge-sequencing context.

Progress

• Reviewer artifacts report passing build, targeted tests, docs validation, git diff --check, and full .agent suite coverage at head 5abc766621aca8c9d115f321fe590d821751dc39.
• Live PR state confirms Normalize agent model footer metadata #407 is intentionally stacked on open PR Normalize Codex GPT-5 answer footer metadata #400; merge Normalize Codex GPT-5 answer footer metadata #400 before Normalize agent model footer metadata #407 or retarget after Normalize Codex GPT-5 answer footer metadata #400 lands.
• Claude’s INFO notes on alias-list drift, duplicated Codex suffix parsing, default-alias fallback coupling, and raw outputs.model are acknowledged as deferrable.
• Existing inline comments and review threads on Normalize agent model footer metadata #407 are empty; no new inline comments were posted, no replies were added, no comments were minimized, and no threads were resolved.

Recommended Next Step

NO_AUTOMATED_ACTION: no unresolved branch-change work remains; humans only need to handle normal stacked-PR merge sequencing.

Final Verdict

SHIP

Action Items

• No required branch-change work remains.

---

codex | gpt-5.5/xhigh | zucchini-m4-6