Concept review - round 2

Round 2 - owner clarification and diagram correction

Date: 2026-10-18. Authority: the existing owner-delegated concept-review scope; owner decisions and correction request are recorded in Sources and evidence. Re-reviewed the affected SYS/SWR, build/browser and detail/bundle/inspection allocations and contracts after applying the remote-media decision. SWR 015 remains an atomic rendered-detail publication responsibility and now explicitly defines the media exception; its SYS 006 parent and SHR 015 amendment agree. No artificial component or duplicate requirement was added. The original round-1 semantic coverage matrix remains valid with media-policy holds removed. Both diagram sources, SVGs and embedded architecture pages were re-reviewed.

STV_REVIEW_009 - Architecture connector routes obscured the flow

Category: visual architecture readability/review omission. Owner: delegated reviewer. State: resolved. Affected: both architecture Draw.io/SVG pairs and their embedded architecture pages. Evidence: the owner identified overlapping arrows; inspection confirmed software GRAPH-to-BUNDLE crossed DETAIL_BUILD, and unconstrained automatic routing allowed flows to coincide or cross boxes. Round 1 checked component/interface coverage and links but missed readability. Impact: readers could not reliably follow sources, destinations and direction. This violates the purpose of a usable architecture flow view despite mechanical link validation passing. Closure criterion: every interface can be followed visually without unrelated-box crossings, coincident runs or hidden labels and arrowheads, at the published reading scale.

Correction: arranged the system flow vertically, reorganized the software components into spaced build/browser flows, fixed connection ports and supplied explicit orthogonal waypoints in dedicated corridors. Retained all 12 software and three system interfaces and all 11 real Need links. Updated embedding aspect ratios to the new exports. SVGs were regenerated by the shared Draw.io exporter; no hand-authored SVG or published raster substitute was used.

Closure evidence: geometry checks found no connector traversing a box interior or sharing a collinear segment with another interface. Actual browser-rendered SVG screenshots and embedded-page views were visually inspected for flow, labels, arrowheads and readable scale. All 11 published Need links were checked again. Temporary screenshots are inspection aids only. Updated concept-creation and its readiness checklist to prevent this defect, and concept-review and its checklist to require actual rendered visual review and hold acceptance when that evidence is missing. The prior review omission is retained here.

Round 2 checks and acceptance

All 71 internal Need records are accepted; four current ADRs are accepted. The 46 stakeholder records remain accepted. No verification_state was promoted. Documentation build with warnings as errors, shared traceability/evidence/CSV validation, published diagram link checks and whitespace checks passed. The two modified shared skills passed quick_validate.py. These checks validate the documentation and concept, not an implemented STV runtime.

Commands: project .venv Python running -m sphinx -E -a -W --keep-going -b html docs docs/_build/html; the shared validate_traceability.py against docs/_build/html/needs.json --version 0.1 --repo-root . --csv docs/_build/traceability.csv; scripts/normalize-diagram-links.py --diagram-dir docs/architecture/diagrams --published-dir docs/_build/html; and git diff --check. Draw.io export ran through the project-local shared-script symbolic link; browser rendering was inspected separately from link validity.

Reviewed inventory - round 2

All current project artifacts below are accepted for concept scope under owner delegation, except governing/shared inputs which are inspected inputs. Runtime verification is not claimed. These hashes supersede the historical round-1 snapshot as the reviewed identities; the review record itself is identified by STV_CONCEPT_REVIEW_002 and is excluded from self-referential hashing.

Source inventory - round 2

Artifact

SHA-256

README.rst

832ba3de77ec144ac73181e852a97f31f1b8031a76d1c66339b40794682db22d

requirements-docs.txt

116d09fd3a50ab57b76508581f7a1740a18ced344da5963d5714fa34d3f7ab8c

build-docs.ps1

aa9e520335f627c96c9b5500806096ea773b2526e713a5f3c2ff25cb5c97b740

docs/conf.py

9cca316d52abcc7ce0de9a2eeda6d35db4bd7ed2f0be557d252f29e3392f0e50

docs/architecture-decisions.rst

178ed52adfd851328eaa36ba619f246ad50d4c1851c06d064a6df996bb311478

docs/architecture-software.rst

99c42caea6a926d912228b84b8ba4b8937cb83eaaa836d231e87f4c87c7ecc4f

docs/architecture-system.rst

71bedf7c61e784d6a5816c78c5cca26021901f77fd339137febcdb1a18cc4a58

docs/elements-software.rst

8c844cd39a128773c0e4f507ef5313e275ba11c5db861c789e43ef4447cefbe2

docs/elements-system.rst

467752106fae5b14d7304985dab373929baa79a2e8af66bec67a0db1358e6ed5

docs/implementation-plan.rst

c2384fca75581e7665ba88c45501703ae78213d32ef7ba26622451fdfe1d5b82

docs/index.rst

7849eb707d5cb0b91ee30f481229afd40e076d6bed6db69e9e6aa0a61046fcda

docs/internal-interfaces.rst

1f24bccbc4ea71dd144f3c666dbd1a9ba2417a9a2feea94af742e9dc4a742d98

docs/method.rst

2c99227c99556525ce0d18ebc62653ab314a0151177130ca9a16a67179a913c0

docs/review.rst

09508f5a2a6a022fd8b93cfd82df85ca3da490a1e5a7613dbc8d2cea6b4a04ab

docs/software.rst

e4a59135dc6a19b850bbdf89cbf3ce4cd155c4a67d1d6a3ff78dda870eae414a

docs/sources.rst

1e993792a3c89039f71d7f066ebf854d61ce9d9dae3f913d11156529bf2cbf47

docs/stakeholder.rst

b43c7329d42eea278af04faf3221e732b83efd985c3d8515d10ef9932eb13a08

docs/system.rst

58b83813962bfcb2e5d63631580bd2a17c56e0a2bad00ba6283cc0e55c4cae04

docs/technology-stack.rst

e766b23f6992cec8602624834b4a45b3c05bc7f93566624c901e350da56b689b

docs/traceability.rst

316d28c29bc4195040eae3af2e91c2412eeba9c34a02da4640a8cef71b28d995

docs/verification-plan.rst

5d1970b05a1c4c9511c6524d5d798e55da9b871c6a4237693458ad3f9daa0793

docs/architecture/decisions/STV_ADR_001_static_extension.rst

dddf0a766caabc294d423e560a15f508bd904679068a1b1cb4511373028e9e73

docs/architecture/decisions/STV_ADR_002_portable_bundle.rst

d5ca127bc6c628a5e033e7a70fbd103b66e288647ab8a67e9b1489e460a902f1

docs/architecture/decisions/STV_ADR_003_state_layout.rst

45fe0d933b22c1e122b15001f895fde75450bc6134b25a0ee40765017fef96ed

docs/architecture/decisions/STV_ADR_004_dependencies.rst

20ee483dbda060b79cebcced32bca2c003d65a10da3b673135d13c067d6572e8

docs/architecture/diagrams/software-architecture.drawio

a952999f9cef4df1996fd0611dd1212f036eda9c386edc605ba2d9a482eaa315

docs/architecture/diagrams/software-architecture.svg

018b59cb2b49870f25f4ce98604ac37dd369f497fbdaa9325795efa334bf2b64

docs/architecture/diagrams/system-architecture.drawio

511fae6bfbd5ed0d8dec26f4838b7933cc009bab147a9d0a6669db487c9b69a1

docs/architecture/diagrams/system-architecture.svg

fe394549a5e26b06a7080389d26762fc30896ec9477daaf9ce6099a20f39d77d

docs/architecture/implementation/check-stack.mjs

97c0f0582660d50ac36efb796098d04e791bb5446049acdac676b9e12f915065

docs/architecture/implementation/package-lock.json

f96e2b0f5c29cee9f69563030caad259ebcafd4a650bd4196560f21bdeb051ba

docs/architecture/implementation/package.json

a7c1aced6534eff63ee4b01475dce5dfdac4d6449fc0d5f8777754ef2daeee60

docs/architecture/implementation/requirements-implementation.txt

e773332c782d0741e1aa07517e85a77cd365be32ff53b851e8baa50e280e6e7e

scripts/export-architecture.ps1

70a249f5423e481f3597d2e288c394b2b45522dc495f40987375a13b2cf788c4

scripts/normalize-diagram-links.py

3a42cc6444f1072e3532012a3daca8a3d5980fd44ebb46a307f42b1875dbf9a8

Shared governing/review inputs (SHA-256):

  • the historical tooling repository/method.md``: 2c62de645f8161149dcdfa777cbd83f03483629c9dc042dcc304dcbded313794.

  • the historical tooling repository/skills/concept-creation/SKILL.md``: c111ab62c5a21f48abb0036732f098c702962b6f6debe7036ccb4d909fbe330f.

  • the historical tooling repository/skills/concept-creation/references/implementation-readiness.md``: adc32423ae34d8634f0e2bc9eb8fbf89ed57a94adbf0cd467935d47dd9e25e55.

  • the historical tooling repository/skills/concept-review/SKILL.md``: f3e4ab6572ae9afbdbaf28c5764937f9254df314857ff9d606c573a53d594fe8.

  • the historical tooling repository/skills/concept-review/references/review-checklist.md``: cca3152ccc94a1e2e5b6e8f39cad1feb38aaafd075d027cbad2b4aa3af0dde86.

Record relocation

The inventory above identifies the reviewed content before the documentation reorganization. Its hashes remain historical evidence. Review commentary and ADR history subsequently moved into central records without changing design obligations or statuses; see Documentation organization and migrated history.

Assessed Need objects

System:

Deliver a self-contained do... (STV_SYS_003), Provide original-object ins... (STV_SYS_006).

Software:

Publish full rendered details (STV_SWR_015), Open canonical object links (STV_SWR_016), Avoid runtime network loading (STV_SWR_017), Use portable output URLs (STV_SWR_018), Constrain and retain detail... (STV_SWR_033), Inspect external Needs with... (STV_SWR_047), Preserve static third-party... (STV_SWR_049), Separate transient hover an... (STV_SWR_050).

Elements System:

Sphinx build integration (STV_SE_BUILD), Browser exploration runtime (STV_SE_BROWSER).

Elements Software:

Rendered detail extraction (STV_SWE_DETAIL_BUILD), Static bundle emission (STV_SWE_BUNDLE), Hover, persistent details a... (STV_SWE_INSPECT).

The diagram correction additionally assessed every element link: Sphinx build integration (STV_SE_BUILD), Browser exploration runtime (STV_SE_BROWSER), Extension lifecycle and dir... (STV_SWE_DIRECTIVE), Host model normalization (STV_SWE_MODEL), Graph adapter and indexes (STV_SWE_GRAPH), Rendered detail extraction (STV_SWE_DETAIL_BUILD), Static bundle emission (STV_SWE_BUNDLE), Exploration state engine (STV_SWE_STATE), Semantic rank layout (STV_SWE_LAYOUT), Accessible view and viewport (STV_SWE_RENDER), Hover, persistent details a... (STV_SWE_INSPECT).