Skip to content

fix(reporting): use host-neutral portable HTML copy - #116

Merged
phodal merged 1 commit into
QoderAI:mainfrom
Cobb04:fix/portable-html-host-neutral-branding
Aug 25, 2026
Merged

fix(reporting): use host-neutral portable HTML copy#116
phodal merged 1 commit into
QoderAI:mainfrom
Cobb04:fix/portable-html-host-neutral-branding

Conversation

@Cobb04

@Cobb04 Cobb04 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

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

  • Spec/ADR, if applicable: No new spec required; Bug [Bug]: Use host-neutral branding in portable HTML reports #115 is sufficient for this narrow presentation-only correction. The completed [Feature]: Qualify DeepSeek Harness for portable HTML report rendering #112 spec remains unchanged at AC-1 through AC-16.
  • Acceptance criteria addressed: host-neutral visible branding and copy; no Codex-directed instruction in non-Codex generated HTML; aligned English/Chinese output; generated HTML regression; Codex and current portable hosts remain valid without per-host renderers.
  • Canonical owners changed: scripts/harness-analysis/renderers/html.mjs, scripts/harness-analysis/renderers/html-interactions.mjs, and their focused generated-output/interaction tests.
  • Explicit non-goals: no layout redesign, renderer architecture rewrite, schema, routing, capability, publication, artifact, Canvas, Studio, dependency, package, version, timeout, or configuration change.

Change Type

  • Feature
  • Bug fix
  • Tests only
  • Documentation/community
  • Refactor with no intended behavior change
  • Dependency, packaging, or infrastructure

Test and Review Evidence

Check Result
Literal default npm run check in a clean detached upstream/main worktree with the exact four-file patch, Node 24.15.0/npm 11.12.1 PASS in 198.87s: root 1,543 passed/2 skipped; Harness 167; Harness UI 30; Harness Studio 270; pack verification 587 npm/849 runtime entries
volta 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) PASS: 2 files, 49 passed, 1 Windows-only skipped
npx vitest run test/reporting/harness-report-render-cli.test.mjs test/reporting/html-report-interactions.test.mjs on Node 24.15.0/npm 11.12.1 PASS: 2 files, 49 passed, 1 Windows-only skipped
Generated validated DSH report.html, English and Chinese PASS: Portable HTML eyebrow/footer and neutral copy-success payload present; legacy Codex-directed visible strings absent
Existing Grok/Kimi/WorkBuddy/Codex HTML routing control PASS: generated artifacts share Portable HTML branding and retain valid routes
git diff upstream/main...HEAD --check PASS
Cobb04 attribution guard before commit, before push, and against the pushed fork range PASS: one Cobb04-authored commit with the required Codex co-author trailer; GitHub resolves the primary author to Cobb04

Manual 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

  • Compatibility and cross-platform impact: visible wording changes intentionally for every portable HTML host; no filesystem or platform behavior changes.
  • Package, plugin, schema, or generated-file impact: none. Existing package contents were verified unchanged apart from the two renderer source files.
  • Rollback or recovery path: revert commit 7e82ca3.
  • Residual risk or unverified boundary: direct local-file screenshot capture is unavailable under the browser security policy; deterministic generated HTML and interaction coverage passed on Node 22 and 24.

AI Involvement

  • Level: Assisted
  • Human review and validation: Cobb04 owns the contribution, directed the independent traceability correction and delivery requirements, and required Issue-first ownership, exact-patch validation, cross-version coverage, skeptical diff review, and attribution verification. AI assistance covered diagnosis, implementation, tests, source/duplicate audit, validation execution, and PR preparation.

Checklist

  • I followed AGENTS.md, CONTRIBUTING.md, and the relevant canonical-owner guidance.
  • The change is focused and does not include unrelated local or generated state.
  • Tests and documentation match the behavior actually delivered.
  • Markdown links were checked when documentation moved or changed. No documentation moved or changed.
  • Cross-platform behavior was considered for Windows, macOS, and Linux.
  • Package/runtime verification was run when shipped files or dependencies changed.
  • User-facing or compatibility changes are recorded in CHANGELOG.md. Not applicable: AGENTS.md prohibits task-external changelog edits without explicit authorization, and Bug [Bug]: Use host-neutral branding in portable HTML reports #115 requires no release metadata change.
  • I have the right to contribute this work under the repository's MIT License.

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>
@phodal
phodal merged commit a415fa6 into QoderAI:main Aug 25, 2026
4 checks passed
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.

[Bug]: Use host-neutral branding in portable HTML reports

2 participants