Repository navigation
fix(docs): add DeepSeek Harness to installation table - #231
Conversation
There was a problem hiding this comment.
Initial maintainer review before the final main synchronization.
Problem: the unescaped double pipe in the DeepSeek Harness Node version constraint was parsed by GitHub Flavored Markdown as table separators, breaking the README layout.
Approach: represent the integration as an explicit three-cell table row, escape the version pipes, and add README and DeepSeek Harness documentation-contract regression coverage.
Impact: documentation and documentation tests only; no runtime, package, or generated artifact changes.
The targeted tests passed 10/10 and the full required CI suite passed. This review was superseded by the final exact-head re-review after the subsequent main synchronization.
There was a problem hiding this comment.
Maintainer re-review after synchronizing with the latest main.
Problem, approach, and impact remain unchanged: fix the malformed DeepSeek Harness README table row with an explicit three-cell row and regression tests. The change is limited to documentation and documentation tests.
Verified exact head 5772283 against main 3411a18. git diff --check passed; the targeted tests passed 10/10; all required CI checks, three-platform package smoke tests, ZIP freshness, and WebM/browser artifact checks passed. No blocking findings.
Problem and value
The DeepSeek Harness installation note was placed immediately after the Installation options table and contained an unescaped Node version disjunction (
||). GitHub Flavored Markdown treated the line as an implicit table row, split the inline code across cells, and rendered a malformed narrow cell with a literal backtick.DeepSeek Harness is an installation surface, so this change represents it as an explicit three-cell row instead of moving it into a long paragraph outside the table.
Scope
README.md,README_EN.md, andREADME_ZH.md.^22.19.0 || >=24.0.0inside one code span.Stability impact
Tests run
cd archify && npm ci— 10 packages installed from the lockfile; 0 vulnerabilities.cd archify && npm test— 1,014 tests; 983 passed, 0 failed, 31 environment-gated tests skipped; golden renders, schemas, generated validators, brand marks, and release identity all passed.node --test archify/test/readme-showcase.test.mjs— 7/7 passed.node --test integrations/deepseek-harness/test/docs-contract.test.mjs— 4/4 passed.node scripts/check-release-identity.mjs—release identity ok: 2.16.0.git diff HEAD^ HEAD --check— passed.Visual evidence
/markdownrender was checked for both English and Chinese READMEs: each contains one DeepSeek Harness<tr>with exactly three<td>cells, the complete||version constraint, and no standalone DeepSeek Harness paragraph.Generated artifacts
None. This changes README Markdown and documentation tests only; Gallery pages, guides, README proof media, and
archify.ziphave no changed authoritative inputs and remain fresh.Checklist
npm testinarchify/.