fix(reporting): use host-neutral portable HTML copy - #116
Merged
phodal merged 1 commit intoAug 25, 2026
Conversation
Replace Codex-specific header, footer, and copy-success text in the shared portable HTML renderer with host-neutral English and Chinese wording. Add generated DSH and cross-host control coverage for Bug QoderAI#115. Validated with the literal npm run check gate on Node 24.15.0 and the focused renderer suites on Node 22.20.0 and Node 24.15.0. Co-authored-by: Codex (GPT 5.6 Sol) <codex@openai.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replace legacy Codex-specific visible text in the shared portable HTML renderer with host-neutral English and Chinese copy. Generated DSH HTML now identifies the format as Portable HTML, and copied fixes direct users to their coding agent rather than Codex.
Why
Independent DSH P0 qualification exposed the shared-renderer bug after #112 / #113 completed. This PR does not reopen or rewrite that historical scope.
Traceability and Scope
scripts/harness-analysis/renderers/html.mjs,scripts/harness-analysis/renderers/html-interactions.mjs, and their focused generated-output/interaction tests.Change Type
Test and Review Evidence
npm run checkin a clean detachedupstream/mainworktree with the exact four-file patch, Node 24.15.0/npm 11.12.1volta run --node 22.20.0 npx vitest run test/reporting/harness-report-render-cli.test.mjs test/reporting/html-report-interactions.test.mjs(npm 10.9.3)npx vitest run test/reporting/harness-report-render-cli.test.mjs test/reporting/html-report-interactions.test.mjson Node 24.15.0/npm 11.12.1report.html, English and Chinesegit diff upstream/main...HEAD --checkManual or visual evidence: temporary English and Chinese DSH reports were generated and passed the self-contained HTML validator. Direct in-app visual capture from
file://was attempted but blocked by the browser's local-file security policy; no bypass or screenshot artifact was used. Generated-output tests exercise header, footer, copy-success text, both locales, non-Codex controls, and Codex validity.Risk and Recovery
7e82ca3.AI Involvement
Checklist
AGENTS.md,CONTRIBUTING.md, and the relevant canonical-owner guidance.CHANGELOG.md. Not applicable:AGENTS.mdprohibits task-external changelog edits without explicit authorization, and Bug [Bug]: Use host-neutral branding in portable HTML reports #115 requires no release metadata change.