Concept review - round 1
Review outcome
Partial acceptance: 8 system requirements, 42 software requirements, six software elements and ADRs 001/003 are accepted under delegation. Two system requirements, eight software requirements and five elements remain reviewed with acceptance held for STV_REVIEW_007. ADRs 002/004 remain proposed. The 46 stakeholder requirements retain their existing owner acceptance and amendment evidence. All verification states remain not_assessed.
The two system elements and all nine software elements were already flagged accepted in the input worktree without a referenced concept review. Their original flags are recorded here as history. Round 1 supplies current evidence for the six clean software elements; the five affected element revisions are reviewed pending the open finding. This is not fabricated historical approval.
Findings and closure evidence
STV_REVIEW_002 - Hover/keyboard detail activation lacked an obligation
Category: missing internal requirement. Owner: delegated reviewer. State: resolved. Affected: STV_SHR_015/034, STV_SYS_006 and STV_SWR_015/033. Evidence: capture and panel geometry were specified, but no SWR explicitly required opening details on hover or keyboard activation. Impact: a structurally covered design could omit the trigger. Correction: added STV_SWR_050 with mixed system derivation and justified keyboard channeling, allocated to INSPECT and RENDER. Closure: reviewed its criterion against the parent intent and added an explicit verification case. Its acceptance is still held by finding 007.
STV_REVIEW_003 - Review summaries and diagnostics were stale
Category: method/content consistency. Owner: delegated reviewer. State: resolved. Affected: stakeholder review notes, STV_SYS_006/010, STV_SWR_028/041, model interface and ADR index. Evidence: previously settled model/title/external-link questions remained described as open; platform checks only had to run rather than pass; a missing-label path was unspecified; ADR summary lacked per-record state/scope/replacement metadata required by the method. Correction: reconciled the recorded decisions, strengthened criteria to actual pass outcomes, specified host-default labels followed by entry-specific errors when still missing, and added the ADR metadata summary. No unapproved owner preference was inferred. Closure: re-read affected text, references and criteria; Sphinx and graph checks validate their mechanical consistency.
STV_REVIEW_004 - Software picture omitted documented interfaces
Category: architecture consistency. Owner: delegated reviewer. State: resolved. Affected: software Draw.io/SVG and component contracts. Evidence: the picture omitted model-to-graph input, graph-to-detail locations and reader commands back to state. Correction: added those three interfaces to the Draw.io source and re-exported through Draw.io using the shared linked scripts. Closure: inspected all nine element boxes and interface edges and validated their published Need anchors. No new unmodeled product component was introduced.
STV_REVIEW_005 - Final detail identity and caching order were incomplete
Category: lifecycle/data contract. Owner: delegated reviewer. State: resolved. Affected: bundle/detail/view contracts, ADR 002 and implementation/verification plans. Evidence: filenames hashed only the Need ID, and pages were written before final rendered detail hashes were available. Impact: changed content could reuse stale browser assets, or a page could name an unknown final manifest. Correction: content-hash detail files, then create the manifest identity from its canonical payload, emit the graph, and finalize only STV-owned tagged script/root attributes in output pages. Cache keys include build identity and expected fragment digest. Current manifests exclude deleted/hidden IDs; publishing a clean tree and the inability to revoke old distributed bundles are explicit. Closure: traced the two-phase dataflow and added cached two-build/failure checks. This is a reviewed design policy, not a claim that its runtime is implemented.
STV_REVIEW_006 - Acceptance flags lacked review scope/evidence
Category: method/acceptance evidence. Owner: delegated reviewer. State: resolved. Affected: both element registers and the concept inventory. Evidence: the input had 11 accepted element flags without a concept-review record while internal requirements/ADRs still lacked review evidence. Correction: recorded the actual owner delegation, input state and current per-artifact outcomes; appended review evidence without changing inferred design basis to observed implementation. Closure: individual acceptance eligibility was checked; blocked revisions are reviewed, not presented as fully accepted. No verification state was promoted.
STV_REVIEW_007 - Remote embedded resources conflict with offline details
Category: owner scope decision. Owner: project owner. State: resolved in round 2. The following original round-1 evidence records why it was open and blocked acceptance of the affected offline/detail path. Affected: STV_SHR_015/017, STV_SYS_003/006, STV_SWR_015/016/017/018/033/047/049/050, STV_SE_BUILD/BROWSER, STV_SWE_DETAIL_BUILD/BUNDLE/INSPECT, ADRs 002/004 and architecture/interface/implementation/verification documents containing that path. Evidence: implementation-plan says to preserve HTTP/HTTPS resource URLs while requiring offline details and no runtime network dependency. A remote image, iframe or CSS resource can trigger an automatic network request when the copied fragment is inserted. The agreed external-object/page fallback does not yet specify remote embedded media inside an otherwise local Need. Impact: either offline operation or full rendered-media parity may be weakened. Recommendation: preserve local bundled media; replace unavailable external embedded media with a clear unavailable-offline indication and an explicit link, with no automatic fetch. An alternative is requiring the host build to provide local cached copies of all such resources. Closure criterion: owner selects the policy; update requirement scope, resource rewriting and fixtures; re-review the affected neighborhood and accept only then. At round 1 the recommendation was not agreed and did not amend the accepted stakeholder obligation. Round-2 closure: the owner explicitly confirmed the recommendation. Updated SHR 015 scope evidence, SWR 015, SYS 006, both resource contracts, ADRs 002/004 and verification fixtures. Re-read the affected offline and inspection paths: local media remains local; uncached remote media becomes a static unavailable-offline indication and explicit resource link. Neither opening details nor extraction downloads remote media. Remote CSS imports and url() resources, SVG, srcset and active embedded documents are covered. The question and documentation correction are both complete; runtime verification is still planned.
STV_REVIEW_008 - Transitive notice list omitted BSD-3-Clause
Category: dependency/packaging completeness. Owner: delegated reviewer. State: resolved. Affected: technology-stack and dependency lock review. Evidence: locked d3-ease 3.0.1 declares BSD-3-Clause, but the notice instructions named only MIT/ISC transitive components. Correction: include BSD-3-Clause in notice requirements and review all locked package metadata. Closure: direct manifest/lock equality and installed-version entries checked; 40 locked package entries include optional platform binaries. Project license selection remains owner-deferred; this finding does not assign a project license.
Semantic coverage and allocation review
All authored SYS/SWR parent links, allocations and software containment were read against their endpoint content. Broad system parents aggregate several obligations; the primary refinements below identify each specific contribution rather than treating a nonempty backlink as sufficient evidence. Supporting state/render components cooperate through the documented interfaces. This is concept coverage, not proof of implementation satisfaction.
Direct channeled obligations address software-local state, presentation or interaction and retain their individual no-system-impact rationale. Mixed-parent SWR 050 ties full inspection to SYS 006 and keyboard details to SHR 034; its software allocations remain inside the system parent’s browser boundary. SWR 048 allocations cover build-side seed, graph and copied-reference filtering. All other SWR allocations match their named component responsibility; all nine software elements belong to the system element in which their allocated behavior executes. No requirement was linked merely to satisfy the graph validator.
Stakeholder |
Internal refinement |
Semantic review |
|---|---|---|
STV_SHR_001 |
SYS 001 / SWR 001 |
RST invocation and source-located instance creation. |
STV_SHR_002 |
SYS 001 / SWR 002, 048 |
Cross-page seeds of any configured visible type; hidden exception retained. |
STV_SHR_003 |
SWR 003 |
One seed and empty expansion keys initially. |
STV_SHR_004 |
SWR 004 |
Nonempty directional plus controls. |
STV_SHR_005 |
SWR 005 |
Complete target-group expansion subject to atomic cap. |
STV_SHR_006 |
SYS 004 / SWR 006 |
Indexed traversal without arbitrary depth cutoff; controls/state provide continuation. |
STV_SHR_007 |
SWR 007, 025 |
Collapse removes the group and recomputes rooted visibility. |
STV_SHR_008 |
SWR 008 |
Complete ID plus two-line title, no field configuration. |
STV_SHR_009 |
SYS 005 / SWR 009 |
Default stakeholder/system/software semantic ranks. |
STV_SHR_010 |
SYS 005 / SWR 010 |
Default system architecture rank between system and software. |
STV_SHR_011 |
SYS 005 / SWR 011 |
Software architecture below software requirements. |
STV_SHR_012 |
SYS 005 / SWR 012 |
Shared tests right of all associated artifacts; later copies scoped separately. |
STV_SHR_013 |
SYS 004 / SWR 013 |
Canonical outgoing links produce incoming adjacency without authored duplicates. |
STV_SHR_014 |
SYS 002 / SWR 014 |
Host schema plus replaceable supplemental mappings. |
STV_SHR_015 |
SYS 006 / SWR 015, 047, 049, 050 |
Capture plus hover/keyboard opening and agreed external/widget scope; media policy held for 007. |
STV_SHR_016 |
SYS 006 / SWR 016 |
Original URI/new-tab action preserves source state; held with inspection path. |
STV_SHR_017 |
SYS 003 / SWR 017 |
Local classic scripts and assets; remote embedded-media policy held for 007. |
STV_SHR_018 |
SYS 003 / SWR 018 |
Relative file/static-host URLs; held with incomplete resource policy. |
STV_SHR_019 |
SWR 019 |
Later in-view occurrence selector, specification accepted without MVP implementation claim. |
STV_SHR_020 |
SWR 020 |
Single identity per object in default shared mode. |
STV_SHR_021 |
SWR 021 |
Later round-trip preservation; later shared-edit policy remains out of MVP scope. |
STV_SHR_022 |
SWR 022 |
Later expansion state keyed independently by occurrence. |
STV_SHR_023 |
SWR 023 |
Later shared union retains independent round-trip state intent. |
STV_SHR_024 |
SWR 024 |
Closing edges/terminal references prevent infinite cycle unfolding. |
STV_SHR_025 |
SWR 025 |
Root reachability retains shared paths and removes disconnected cycles. |
STV_SHR_026 |
SYS 001 / SWR 026 |
Missing/unknown seed errors include invocation source. |
STV_SHR_027 |
SYS 004 / SWR 027 |
Host owns broken-trace failure; valid graph remains renderable. |
STV_SHR_028 |
SYS 004 / SWR 028 |
Missing ID/text/connection, no false original link; hidden targets excluded. |
STV_SHR_029 |
SWR 029 |
No controls for empty groups; isolated-object indication. |
STV_SHR_030 |
SWR 030 |
Host colors and consistent built-in fallback palette. |
STV_SHR_031 |
SWR 031 |
Red border outside accepted set, including empty/missing status. |
STV_SHR_032 |
SWR 032 |
Accepted-status override with accepted default. |
STV_SHR_033 |
SYS 006 / SWR 033 |
Viewport bounds, pointer retention and scrolling; held with detail path. |
STV_SHR_034 |
SWR 034, 050 |
Keyboard controls, focus and detail activation/dismissal; media-specific component held. |
STV_SHR_035 |
SWR 035 |
Separate independently keyed controls with host directional labels. |
STV_SHR_036 |
SYS 002 / SWR 036 |
Entry-specific validation failure with agreed optional defaults. |
STV_SHR_037 |
SYS 005 / SWR 037 |
Authoritative ordered model levels, not graph-depth ranks. |
STV_SHR_038 |
SYS 007 / SWR 038 |
Independent state/viewport per view; later selectors conditional. |
STV_SHR_039 |
SWR 039 |
Pointer/keyboard zoom changes transform, not graph state. |
STV_SHR_040 |
SWR 040 |
Pointer/keyboard pan reaches off-screen graph content. |
STV_SHR_041 |
SYS 010 / SWR 041 |
Actual passing workflow results on recorded stable Firefox/Windows. |
STV_SHR_042 |
SYS 009 / SWR 042 |
Clean wheel on pinned host and minimum Python 3.10 check. |
STV_SHR_043 |
SYS 008 / SWR 043 |
200 visible occurrences including placeholders; benchmark budgets deliberately later. |
STV_SHR_044 |
SYS 008 / SWR 044 |
Atomic rejected transition with explanation and unchanged state. |
STV_SHR_045 |
SYS 002 / SWR 045 |
Reuse host types/colors/labels without a duplicate registry. |
STV_SHR_046 |
SYS 009 / SWR 046 |
Installable package setup/assets for an independent host. |
Checks and remaining conditions
Round 1 checks include warnings-as-errors Sphinx, shared graph validation with repository-relative evidence and CSV output, published Draw.io/SVG Need links, manifest/lock equality and dependency metadata, installed host API/lifecycle source inspection, and actual semantic reading of the inventory above. Exact result summary is recorded after the round’s final build below. Primary host reference inspected: Sphinx event lifecycle. Installed Sphinx-Needs 8.2.0 registers write-started postprocessing before the proposed priority 800 handler and lock release at build-finished priority 9999.
Original library smoke-check and extraction results remain dated historical evidence in Sources and evidence; this review does not reclassify them as runtime tests. No product code is implemented. Deferred license selection, measured budgets and later occurrence-mode policy are recorded conditions, not hidden findings or falsely passed checks. The open media decision prevents full concept acceptance.
Artifact inventory and content identities
Each row records the reviewed source content at the end of round 1. SHA-256 identifies the historical reviewed revision; acceptance scope is limited as stated. The review record itself is identified by STV_CONCEPT_REVIEW_001/round 1 and retains its issue history rather than attempting a self-referential file hash. The live method is a governing input, not a newly accepted project document.
Governing method SHA-256: 2c62de645f8161149dcdfa777cbd83f03483629c9dc042dcc304dcbded313794.
Artifact |
Review outcome |
SHA-256 |
|---|---|---|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Held for STV_REVIEW_007 |
|
|
Held for STV_REVIEW_007 |
|
|
Per-record accepted/reviewed; see outcome |
|
|
Per-record accepted/reviewed; see outcome |
|
|
Held for STV_REVIEW_007 |
|
|
Accepted concept artifact; no runtime claim |
|
|
Held for STV_REVIEW_007 |
|
|
Shared governing/tool input, inspected |
|
|
Current review/evidence register |
|
|
Per-record accepted/reviewed; see outcome |
|
|
Current review/evidence register |
|
|
Owner-accepted baseline, reviewed as input |
|
|
Per-record accepted/reviewed; see outcome |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Held for STV_REVIEW_007 |
|
|
Accepted concept artifact; no runtime claim |
|
|
Held for STV_REVIEW_007 |
|
|
Accepted concept artifact; no runtime claim |
|
|
Held for STV_REVIEW_007 |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Accepted concept artifact; no runtime claim |
|
|
Shared governing/tool input, inspected |
|
|
Shared governing/tool input, inspected |
|
Round 1 mechanical results: Sphinx warnings-as-errors passed; graph/evidence validation passed (46 SHR, 10 SYS, 50 SWR, two system and nine software elements); CSV export passed; 11 diagram Need links passed; source whitespace check passed. Status audit confirms 46 existing accepted stakeholders, 56 accepted internal Need records and 15 reviewed internal records. ADRs 001/003 are accepted; 002/004 remain proposed. No verification_state changed from not_assessed.
The architecture diagrams are accepted for the responsibilities and connections they depict; they do not select the unresolved remote-resource policy. Likewise, the technology stack is accepted as a dependency/packaging selection, not as proof that every feature path has been accepted. Held parent/view documents retain the explicit unresolved policy in their review scope.
Assessed Need objects
Stakeholder:
Embed a traceability view (STV_SHR_001), Select the starting object (STV_SHR_002), Initially collapse traced o... (STV_SHR_003), Offer expansion by traceabi... (STV_SHR_004), Reveal related objects (STV_SHR_005), Continue exploration (STV_SHR_006), Collapse opened traces (STV_SHR_007), Inspect discovered objects (STV_SHR_008), Preserve requirement levels (STV_SHR_009), Position system architecture (STV_SHR_010), Position software architecture (STV_SHR_011), Position test evidence in c... (STV_SHR_012), Navigate relationships in b... (STV_SHR_013), Use a replaceable input dat... (STV_SHR_014), Inspect object details on h... (STV_SHR_015), Open the original Need obje... (STV_SHR_016), Explore documentation offline (STV_SHR_017), Explore documentation on st... (STV_SHR_018), Select how shared objects a... (STV_SHR_019), Default to shared nodes (STV_SHR_020), Preserve exploration when s... (STV_SHR_021), Explore separate occurrence... (STV_SHR_022), Combine expanded traces in ... (STV_SHR_023), Show cycles without repeati... (STV_SHR_024), Preserve objects reached th... (STV_SHR_025), Require a valid starting ID (STV_SHR_026), Render valid traces despite... (STV_SHR_027), Identify unresolved trace t... (STV_SHR_028), Indicate objects without re... (STV_SHR_029), Distinguish artifact types ... (STV_SHR_030), Mark artifacts that are not... (STV_SHR_031), Configure accepted-status v... (STV_SHR_032), Keep long tooltips readable... (STV_SHR_033), Operate the view using the ... (STV_SHR_034), Distinguish incoming and ou... (STV_SHR_035), Reject invalid or incomplet... (STV_SHR_036), Define additional ordered a... (STV_SHR_037), Explore multiple views inde... (STV_SHR_038), Zoom the traceability view (STV_SHR_039), Pan the traceability view (STV_SHR_040), Support Firefox desktop (STV_SHR_041), Support the initial documen... (STV_SHR_042), Explore a bounded view of a... (STV_SHR_043), Explain expansions blocked ... (STV_SHR_044), Reuse the host project's mo... (STV_SHR_045), Deliver a reusable installa... (STV_SHR_046).
System:
Author and resolve embedded... (STV_SYS_001), Integrate the host data model (STV_SYS_002), Deliver a self-contained do... (STV_SYS_003), Expose the complete host tr... (STV_SYS_004), Preserve semantic layout (STV_SYS_005), Provide original-object ins... (STV_SYS_006), Isolate view sessions (STV_SYS_007), Bound exploration of large ... (STV_SYS_008), Deliver the reusable extension (STV_SYS_009), Support the initial executi... (STV_SYS_010).
Software:
Register the RST entry point (STV_SWR_001), Resolve seeds after collection (STV_SWR_002), Initialize collapsed state (STV_SWR_003), Expose plus controls (STV_SWR_004), Expand complete target groups (STV_SWR_005), Traverse without depth cutoff (STV_SWR_006), Collapse relationship groups (STV_SWR_007), Render fixed node summaries (STV_SWR_008), Order requirement ranks (STV_SWR_009), Insert system architecture ... (STV_SWR_010), Place software architecture... (STV_SWR_011), Place classified test evide... (STV_SWR_012), Index both trace directions (STV_SWR_013), Normalize replacement mappings (STV_SWR_014), 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), Offer occurrence mode later (STV_SWR_019), Use shared identity initially (STV_SWR_020), Retain state across later m... (STV_SWR_021), Key later expansions by occ... (STV_SWR_022), Project later shared expans... (STV_SWR_023), Close repeated paths safely (STV_SWR_024), Compute seed-rooted visibility (STV_SWR_025), Report invalid invocation s... (STV_SWR_026), Tolerate host broken traces (STV_SWR_027), Create missing-target nodes (STV_SWR_028), Indicate empty adjacency (STV_SWR_029), Resolve type palette (STV_SWR_030), Apply acceptance borders (STV_SWR_031), Normalize accepted values (STV_SWR_032), Constrain and retain detail... (STV_SWR_033), Provide keyboard interaction (STV_SWR_034), Label directional controls (STV_SWR_035), Reject invalid model entries (STV_SWR_036), Honor resolved declared lev... (STV_SWR_037), Scope state to view roots (STV_SWR_038), Implement view-local zoom (STV_SWR_039), Implement view-local pan (STV_SWR_040), Verify Firefox on Windows (STV_SWR_041), Package for the pinned tool... (STV_SWR_042), Enforce visible occurrence ... (STV_SWR_043), Reject over-limit transitio... (STV_SWR_044), Reuse canonical host defini... (STV_SWR_045), Expose an installable entry... (STV_SWR_046), Inspect external Needs with... (STV_SWR_047), Respect host-hidden Need ob... (STV_SWR_048), 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:
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).