@oevery/rsp 4.0.0 → 4.2.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/chunk-AUZYUJ46.mjs +418 -0
- package/dist/chunk-DNRGBQS7.mjs +108 -0
- package/dist/chunk-LGDCIQYL.mjs +725 -0
- package/dist/chunk-P3KO3GZW.mjs +1023 -0
- package/dist/{chunk-HFSTLCCK.mjs → chunk-VDRXFKBP.mjs} +133 -1800
- package/dist/cli.mjs +286 -294
- package/dist/client-U2PN3VS4.mjs +108 -0
- package/dist/{entry-RIJQ2X46.mjs → entry-NDASTAI5.mjs} +13 -9
- package/dist/viewer/NOTICE.txt +299 -0
- package/dist/viewer/app.css +2 -0
- package/dist/viewer/app.js +670 -0
- package/dist/viewer/chunks/abnfDiagram-S7VZAUFO-K5EOZOEF.js +1 -0
- package/dist/viewer/chunks/architecture-WOLXFQ4H-UAT5D5BF.js +1 -0
- package/dist/viewer/chunks/architectureDiagram-YGSQEFNW-UNPSYM2L.js +36 -0
- package/dist/viewer/chunks/blockDiagram-APKKGZFW-2F7TTPAV.js +140 -0
- package/dist/viewer/chunks/bpmnIcons-APHXDYXF-ZWPFFN3O.js +1 -0
- package/dist/viewer/chunks/c4Diagram-KJ2TOQU5-KNLN6CME.js +38 -0
- package/dist/viewer/chunks/chunk-274PZN6B.js +1 -0
- package/dist/viewer/chunks/chunk-2VGDQLQD.js +1 -0
- package/dist/viewer/chunks/chunk-33XESOCS.js +1 -0
- package/dist/viewer/chunks/chunk-3ZSL2TGQ.js +1 -0
- package/dist/viewer/chunks/chunk-432I4U3B.js +1 -0
- package/dist/viewer/chunks/chunk-4PJNGS3V.js +1 -0
- package/dist/viewer/chunks/chunk-54NLSLTF.js +9 -0
- package/dist/viewer/chunks/chunk-6QRHYJ5Q.js +1 -0
- package/dist/viewer/chunks/chunk-6RCFKBVR.js +1 -0
- package/dist/viewer/chunks/chunk-6Y2HP7DC.js +213 -0
- package/dist/viewer/chunks/chunk-7MAT7NHP.js +1 -0
- package/dist/viewer/chunks/chunk-AKHBD6LT.js +2 -0
- package/dist/viewer/chunks/chunk-CZIM56BV.js +1 -0
- package/dist/viewer/chunks/chunk-DN4Y7LXJ.js +1 -0
- package/dist/viewer/chunks/chunk-E2UQN267.js +1 -0
- package/dist/viewer/chunks/chunk-GBARLDDT.js +1 -0
- package/dist/viewer/chunks/chunk-H4YEV7VR.js +1 -0
- package/dist/viewer/chunks/chunk-HUVVDQHZ.js +1 -0
- package/dist/viewer/chunks/chunk-IAKC32WV.js +72 -0
- package/dist/viewer/chunks/chunk-JTW4SY4L.js +1 -0
- package/dist/viewer/chunks/chunk-JZV675LF.js +1 -0
- package/dist/viewer/chunks/chunk-KDUDYTWO.js +1 -0
- package/dist/viewer/chunks/chunk-KHX3A5RZ.js +331 -0
- package/dist/viewer/chunks/chunk-KPXJJSUR.js +15 -0
- package/dist/viewer/chunks/chunk-LEGNM6PH.js +1 -0
- package/dist/viewer/chunks/chunk-LYY4ZY72.js +1 -0
- package/dist/viewer/chunks/chunk-MKDEK2LU.js +1 -0
- package/dist/viewer/chunks/chunk-MYJRFART.js +129 -0
- package/dist/viewer/chunks/chunk-NUXZBURC.js +1 -0
- package/dist/viewer/chunks/chunk-NXHT5I2L.js +1 -0
- package/dist/viewer/chunks/chunk-OF62XRVM.js +62 -0
- package/dist/viewer/chunks/chunk-ONCOEVBS.js +1 -0
- package/dist/viewer/chunks/chunk-P44SH3OO.js +1 -0
- package/dist/viewer/chunks/chunk-PDXKDBIX.js +1 -0
- package/dist/viewer/chunks/chunk-PHKIAKHS.js +1 -0
- package/dist/viewer/chunks/chunk-PJDMAGPE.js +2 -0
- package/dist/viewer/chunks/chunk-RC2NPHSX.js +88 -0
- package/dist/viewer/chunks/chunk-T6DM5RSK.js +10 -0
- package/dist/viewer/chunks/chunk-U5STX66M.js +2 -0
- package/dist/viewer/chunks/chunk-UIAJQPOF.js +1 -0
- package/dist/viewer/chunks/chunk-UX2S3LP3.js +1 -0
- package/dist/viewer/chunks/chunk-VEMTQQ7Y.js +2 -0
- package/dist/viewer/chunks/chunk-WKCPBR3C.js +1 -0
- package/dist/viewer/chunks/chunk-WMQ4Y2MS.js +32 -0
- package/dist/viewer/chunks/chunk-YPWIQAHX.js +3 -0
- package/dist/viewer/chunks/chunk-Z4ZDIMO2.js +136 -0
- package/dist/viewer/chunks/chunk-ZATQCNXH.js +1 -0
- package/dist/viewer/chunks/chunk-ZE5RDJX6.js +1 -0
- package/dist/viewer/chunks/chunk-ZO62R4L2.js +1 -0
- package/dist/viewer/chunks/classDiagram-v2-L25IHNXF-5F2M2CZ4.js +217 -0
- package/dist/viewer/chunks/cose-bilkent-JH36ORCC-S56HJ5WN.js +1 -0
- package/dist/viewer/chunks/cynefin-EF2NZ3EQ-6WY2NCLE.js +1 -0
- package/dist/viewer/chunks/cynefinDiagram-L36ESFBY-DE6PZNQA.js +62 -0
- package/dist/viewer/chunks/dagre-SPEB5RXH-QEOTKFV2.js +4 -0
- package/dist/viewer/chunks/diagram-3B7GL73J-RZPIU7FY.js +200 -0
- package/dist/viewer/chunks/diagram-DNEMPZML-YD26TJKC.js +24 -0
- package/dist/viewer/chunks/diagram-L3MUGCDN-TJUEX4QI.js +41 -0
- package/dist/viewer/chunks/diagram-LA4NIPWU-3PUQS6UW.js +24 -0
- package/dist/viewer/chunks/diagram-MHCOC4E2-YBTBD4PG.js +3 -0
- package/dist/viewer/chunks/diagram-XPFHEONP-VF2TQBNR.js +30 -0
- package/dist/viewer/chunks/ebnfDiagram-UZSYDL2S-QE7DIEME.js +1 -0
- package/dist/viewer/chunks/elk-XV6XMXQE-X5OZRIL3.js +27 -0
- package/dist/viewer/chunks/erDiagram-EYBAPDEH-5B2S7VB4.js +99 -0
- package/dist/viewer/chunks/eventmodeling-K75KTNOO-CHCQYZBI.js +1 -0
- package/dist/viewer/chunks/flowDiagram-MALGAFMO-XJFC5NTN.js +1 -0
- package/dist/viewer/chunks/ganttDiagram-23REXITY-TUUZ3QYO.js +292 -0
- package/dist/viewer/chunks/gitGraph-VSP46ZUC-VFZPJQ6A.js +1 -0
- package/dist/viewer/chunks/gitGraphDiagram-STJD7YCW-4CVV5LSH.js +106 -0
- package/dist/viewer/chunks/info-OHQRW6UA-P4744SJ6.js +1 -0
- package/dist/viewer/chunks/infoDiagram-KBXO4MBG-Q2WYYNNN.js +2 -0
- package/dist/viewer/chunks/ishikawaDiagram-M35NCXHK-PLQPTX2U.js +70 -0
- package/dist/viewer/chunks/journeyDiagram-OPLOXDXO-FB73RE6F.js +139 -0
- package/dist/viewer/chunks/kanban-definition-BZKZHSQY-6FU4TXYJ.js +89 -0
- package/dist/viewer/chunks/katex-IN4S3RHS.js +257 -0
- package/dist/viewer/chunks/mindmap-definition-I2QAS2VK-ZQ36ABVW.js +98 -0
- package/dist/viewer/chunks/packet-JDAUHWVQ-NYXIQQVA.js +1 -0
- package/dist/viewer/chunks/pegDiagram-FD4XPPHP-5XXVT6XN.js +1 -0
- package/dist/viewer/chunks/pie-XZMESJXO-32CUE6KZ.js +1 -0
- package/dist/viewer/chunks/pieDiagram-J4AKIIQI-WRTBDFM5.js +39 -0
- package/dist/viewer/chunks/quadrantDiagram-XTYBLID2-YPBDQIAK.js +7 -0
- package/dist/viewer/chunks/radar-ABXABTNO-272FILRO.js +1 -0
- package/dist/viewer/chunks/railroad-I3PHUGI6-6EGAH4XA.js +1 -0
- package/dist/viewer/chunks/railroad-abnf-LNEFI6M7-G6E7ATER.js +1 -0
- package/dist/viewer/chunks/railroad-ebnf-DWYD2UWJ-MR4P6BDR.js +1 -0
- package/dist/viewer/chunks/railroad-peg-JR7BVNR7-WXLQUGUO.js +1 -0
- package/dist/viewer/chunks/railroadDiagram-MRQ6NCDL-XHZQ2BLY.js +1 -0
- package/dist/viewer/chunks/requirementDiagram-Y7LWUY3S-W5FHRRML.js +84 -0
- package/dist/viewer/chunks/sankeyDiagram-WGJFYUXA-OJJIRVTI.js +40 -0
- package/dist/viewer/chunks/sequenceDiagram-5NDRBG5M-D6T4DRNG.js +169 -0
- package/dist/viewer/chunks/sizeCapture-INFHLROL-WL3FTUJX.js +1 -0
- package/dist/viewer/chunks/stateDiagram-v2-FJ4FJNEA-LAXL7KGG.js +291 -0
- package/dist/viewer/chunks/swimlanes-UMQ2AFYF-XIJBD726.js +1 -0
- package/dist/viewer/chunks/swimlanesDiagram-OMPLPG4O-7O7MHFW7.js +8 -0
- package/dist/viewer/chunks/timeline-definition-SEUHKTQE-BAZQA47I.js +120 -0
- package/dist/viewer/chunks/treeView-D4JQ5SDB-6YMDAAPA.js +1 -0
- package/dist/viewer/chunks/treemap-SAJKECNS-2X573PPS.js +1 -0
- package/dist/viewer/chunks/usecaseDiagram-AFA6YXN4-4YCEMWJB.js +370 -0
- package/dist/viewer/chunks/vennDiagram-NNMJR3RB-GEMROLAE.js +34 -0
- package/dist/viewer/chunks/wardley-7MLQ67FV-DGTGHYYC.js +1 -0
- package/dist/viewer/chunks/wardleyDiagram-S7YFUSEH-K4HVD3SS.js +78 -0
- package/dist/viewer/chunks/xychartDiagram-GGNKY7IX-HVLWCUEO.js +7 -0
- package/dist/viewer/mermaid.js +12 -0
- package/dist/viewer-build.json +1 -0
- package/dist/viewer-service.mjs +607 -0
- package/package.json +27 -8
- package/skills/rsp/SKILL.md +13 -10
- package/skills/rsp/references/architecture.md +39 -0
- package/skills/rsp/references/closeout.md +4 -4
- package/skills/rsp/references/durable-review.md +6 -9
- package/skills/rsp/references/focus-continuation.md +6 -2
- package/skills/rsp/references/setup-repair.md +11 -6
- package/skills/rsp/references/work-model.md +14 -4
- package/skills/rsp-commit/SKILL.md +2 -2
- package/skills/rsp-doc/SKILL.md +13 -10
- package/skills/rsp-doc/references/skill-authoring.md +2 -2
- package/skills/rsp-implement/SKILL.md +16 -12
- package/skills/rsp-review/SKILL.md +12 -12
- package/skills/rsp-review/references/code-review.md +11 -5
- package/skills/rsp-review/references/document-review.md +8 -10
- package/skills/rsp-shape/SKILL.md +3 -1
- package/skills/rsp-shape/references/planning.md +4 -4
- package/skills/rsp-verify/SKILL.md +9 -14
- package/skills/rsp-doc/references/artifact-lenses.md +0 -43
- package/skills/rsp-review/references/structural-review.md +0 -41
|
@@ -4,7 +4,7 @@ description: Judge a fixed change or file set, or discover evidenced structural
|
|
|
4
4
|
license: MIT
|
|
5
5
|
metadata:
|
|
6
6
|
author: oevery
|
|
7
|
-
version: "2026.
|
|
7
|
+
version: "2026.11.04.1"
|
|
8
8
|
---
|
|
9
9
|
|
|
10
10
|
# RSP Review
|
|
@@ -29,11 +29,9 @@ The user request fixes the requested outcome and allowed operations subject to n
|
|
|
29
29
|
|
|
30
30
|
## Select perspectives before loading
|
|
31
31
|
|
|
32
|
-
|
|
32
|
+
Select methods from the judged content and its risks. Two perspectives require neither two reports nor two reviewers. Supporting references inform the judgment without becoming additional review targets. Respect narrower requested scope and reuse applicable instructions already in context.
|
|
33
33
|
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
- **[Code](references/code-review.md):** behavior, contracts, safety and failures of reviewed executable instructions/code, runnable recipes, algorithmic logic or tool-interpreted content. A prose behavior claim alone can remain Document grounding.
|
|
34
|
+
- **[Code](references/code-review.md):** behavior, contracts, safety and failures of executable content; also ownership, dependency and verification lenses for explicitly bounded structural discovery. Within change review, structural inspection stays with a concrete risk in the fixed boundary. A prose behavior claim alone can remain Document grounding.
|
|
37
35
|
- **[Document](references/document-review.md):** factual explanation, usability, navigation and maintenance of reviewed content, including API/TSDoc comments, requirements and human- or agent-facing guidance.
|
|
38
36
|
- Mixed targets receive both applicable perspectives; complete Skill instruction reviews normally need both. Ordinary implementation comments need local accuracy/clarity checks, not an automatic full Document pass; bounded wording changes need only affected perspectives. Illustrative pseudocode needs logical consistency, not an invented runtime or production test gate.
|
|
39
37
|
|
|
@@ -41,9 +39,11 @@ When fixed reviewed artifacts include changelogs, release notes, migration guide
|
|
|
41
39
|
|
|
42
40
|
## Inspect the fixed scope
|
|
43
41
|
|
|
44
|
-
Inspect checkout state and the selected diff or structural boundary, then authority, then the smallest behavior chain and evidence needed for the question.
|
|
42
|
+
Inspect checkout state and the selected diff or structural boundary, then authority, then the smallest behavior chain and evidence needed for the question. Report a comment/implementation or example/contract discrepancy once, with evidence from both sides; presume neither correct. Stop when applicable judgments are supported. Do not broaden scope or execute snippets without authority.
|
|
43
|
+
|
|
44
|
+
Reuse traceable check evidence while its tested scope and relevant code, dependencies, configuration, tools, inputs and environment still apply. Before running an authorized check, identify the evidence gap, invalidated result, changed risk or explicit execution requirement it addresses. Apply this selection to tests and non-test checks alike; low cost alone does not justify extra execution. Distinguish reused evidence from new observations.
|
|
45
45
|
|
|
46
|
-
|
|
46
|
+
When the reviewed scope affects modeled structure, responsibilities, dependencies or bindings, apply [architecture knowledge](../rsp/references/architecture.md) within that scope.
|
|
47
47
|
|
|
48
48
|
For re-review, inspect the actual correction and affected behavior against the original scope, expanding within it for evidenced consequences. Earlier clean verdicts do not establish a changed target; unchanged tests need no rerun solely for a new judgment. Perform the requested semantic review: neither passing tests nor an implementer summary supplies that judgment or required independence.
|
|
49
49
|
|
|
@@ -53,21 +53,21 @@ Missing comparison or authority blocks dependent pipelines; other gaps follow th
|
|
|
53
53
|
|
|
54
54
|
## Report
|
|
55
55
|
|
|
56
|
-
Responses follow explicit language instruction, personal instructions, then conversation language, never project artifact-language settings. Preserve paths, commands, identifiers, WorkOwner/WorkRefs
|
|
56
|
+
Responses follow explicit language instruction, personal instructions, then conversation language, never project artifact-language settings. Preserve paths, commands, identifiers, WorkOwner/WorkRefs and `P0`-`P3`; use localized narration with differing machine values as secondary exact tokens. Use `blocked | findings | clean` for the overall verdict. For per-file or per-perspective results, preserve the caller's required schema, including `issues_found` or `skipped` where specified.
|
|
57
57
|
|
|
58
58
|
Preserve the information below without mandatory headings or one line per field. Use a compact report for a clean bounded review, omit empty Findings and unrelated no-op fields, and honor explicitly required formats:
|
|
59
59
|
|
|
60
60
|
- Scope: WorkOwner or direct request, fixed comparison, authoritative intent, applicable perspectives and material exclusions.
|
|
61
|
-
- Findings when present: severity/title,
|
|
61
|
+
- Findings when present: severity/title, changed lines or the smallest inspected chain, evidence and consequence, and the smallest correction or unresolved owner decision. For a defect, show its reachable trigger; for a simplification, show the current responsibility, demonstrated burden and smaller complete alternative preserving failure, authority and Required gates. Cite authority when a contract is violated and state material uncertainty. Report one root issue once even when both perspectives are affected.
|
|
62
62
|
- Coverage: each perspective's checked content, material omissions and unverified scope, including non-actionable gaps; distinguish reused evidence from new observations.
|
|
63
63
|
- Verdict: `blocked | findings | clean`; give the next action only when an issue, blocker or requested follow-up remains. Do not repeat the findings or evidence.
|
|
64
64
|
|
|
65
|
-
Use `clean` only for an applicable judgment with no actionable issue, `skipped` for an inapplicable perspective when the requested format requires it, and `blocked` when required scope, authority or decisive evidence is missing. Findings
|
|
65
|
+
Use `clean` only for an applicable judgment with no actionable issue, `skipped` for an inapplicable perspective when the requested format requires it, and `blocked` when required scope, authority or decisive evidence is missing. Findings may establish a defect, an explicit-requirement violation, or an evidenced simplification; fewer lines alone, speculative impact and taste-only cleanup do not qualify. Do not invent a wrong-result bug for a supported simplification. Missing evidence alone is a coverage limit. Retain supported findings while blocking only affected judgments.
|
|
66
66
|
|
|
67
|
-
Order findings by severity and path: P0 critical security/data/breakage risk; P1 normal-path contract failure; P2 meaningful edge/maintenance risk; P3 narrow actionable improvement.
|
|
67
|
+
Order findings by severity and path: P0 critical security/data/breakage risk; P1 normal-path contract failure; P2 meaningful edge/maintenance risk; P3 narrow actionable improvement. Assess findings by consequence and confidence, not lines removed or ease of correction.
|
|
68
68
|
|
|
69
69
|
Illustrative compact finding, not an observed defect:
|
|
70
70
|
|
|
71
|
-
> [P1] Failure is reported as success — `src/save.ts:42
|
|
71
|
+
> [P1] Failure is reported as success — `src/save.ts:42`. The selected Spec requires rejection, but this catch returns success after a failed write, so callers acknowledge unsaved data. Preserve rejection and verify the failure path.
|
|
72
72
|
|
|
73
73
|
Return a report only. Do not edit files, apply fixes, change focus, create RSP artifacts, switch branches, stage, commit, push, open a PR, publish, delete, trigger external review, or approve. Later actions require separate explicit authority.
|
|
@@ -21,10 +21,16 @@ Check in this order:
|
|
|
21
21
|
- Compare each changed Code artifact's public return and failure behavior with the fixed baseline. Changes between throw/rejection, sentinel, `null`, status code or result object are failure-contract changes even when intended. Require focused tests or other explicit verification for the affected judgment. If decisive evidence is unavailable, report the gap and block that judgment under the shared report contract. Emit a Finding for a demonstrated defect or an evidenced violation of an explicit project/task verification requirement, not absence of a new test alone.
|
|
22
22
|
- Simple deterministic correction is excepted when public behavior shape is preserved and no risky branch, state transition, concurrency, persistence, security or failure delivery changes, despite a corrected value. Failure-contract changes are never excepted.
|
|
23
23
|
6. **Test and seam value — hard gate before `clean`:**
|
|
24
|
-
- For
|
|
25
|
-
-
|
|
26
|
-
-
|
|
27
|
-
- Report test-only generality, wrapper/forwarding assertions, self-confirming expectations, source-string coupling or duplicate failure coverage only with a concrete real owner or smaller evidence path. Keep helpers clear about important inputs and assertions; useful shared setup and limited repetition are both acceptable. Tests protecting independent consequences retain value; fewer tests are not inherently better.
|
|
24
|
+
- For a questioned test or added boundary, identify its real consumer, distinct observable consequence and evidence gap. Tests/docs alone establish no production load. Prefer stable public behavior and real internal collaboration; substitute external or nondeterministic boundaries as needed. Meaningful algorithms, parsers and critical boundaries can warrant focused unit tests.
|
|
25
|
+
- Derive expectations from the contract or independent examples, not the tested computation or source spelling. Check uncertain fixture conditions with targeted evidence. Pin details, counts or ordering only when contract-relevant. Private structure and constants compared with themselves are not behavioral contracts; a public default can be.
|
|
26
|
+
- Report duplicate confidence, forwarding-only assertions or test-only generality with a concrete smaller evidence path. Keep important scenario inputs and assertions visible. Independent consequences retain value; neither per-function/file/branch allocation nor a unit/integration ratio determines coverage.
|
|
28
27
|
7. **Simplicity:** for unnecessary abstraction, duplication, indirection, dependency or scope expansion, identify a smaller alternative preserving required behavior. Check who owns each rule, whether ordinary changes stay local and what internals callers must know. Similar code with different responsibilities may stay separate; unrelated flags can increase coupling through forced reuse. Judge extraction or removal by caller and maintainer burden, not file size, layer count or reuse count.
|
|
29
28
|
|
|
30
|
-
|
|
29
|
+
## Bounded structural discovery
|
|
30
|
+
|
|
31
|
+
Use only for a requested repository/subtree audit or a concrete structural risk within fixed-change scope. Without a narrower lead, use recurring changes or defects to select a candidate chain; history is navigation evidence, not a finding. Inspect current owners and the live path. Do not turn these lenses into a whole-repository checklist:
|
|
32
|
+
|
|
33
|
+
- **Ownership:** trace creation, validation, mutation, persistence, recovery and consumption. Report competing authority only when reachable paths can diverge with a concrete consequence; projections and fixtures are not automatically competing owners.
|
|
34
|
+
- **Dependencies:** start from a consumer's required knowledge of provider policy, representation, ordering, failures and lifecycle. Show actual change amplification or invalid-state exposure, not merely an import or large file. Trace a proposed layer removal: eliminating pass-through work may help, but scattering policy or failure knowledge into callers does not.
|
|
35
|
+
- **Repeated knowledge:** trace one representative change across its independently maintained owners. Generated copies, deliberate boundary translation and similar text alone establish no duplication problem.
|
|
36
|
+
- **Verification:** map claimed evidence to the same entry path and public result. A bypassed isolated seam can give false confidence; missing tests alone establish neither a structural finding nor a functional defect.
|
|
@@ -4,7 +4,7 @@ Load for explanation or usage guidance within the fixed reviewed content, includ
|
|
|
4
4
|
|
|
5
5
|
- Compare API comments and copyable examples with the smallest authoritative contract and implementation: inputs, outputs, failures, prerequisites and consequences.
|
|
6
6
|
- Label illustrative pseudocode and incomplete snippets; neither is a verified runnable recipe. A code fence or evidence-source read does not select Code by itself, and review grants no execution authority.
|
|
7
|
-
- Code handles applicable behavior/tool checks, not a duplicate writing pass.
|
|
7
|
+
- Code handles applicable behavior/tool checks, not a duplicate writing pass.
|
|
8
8
|
- Skill guidance: descriptions discriminate neighboring intents; actions have prerequisites/results and conditional links have a reading purpose. Keep common safety and minimum actionable context local. Tables serve short stable mappings, not branching procedures; concise wording preserves substantive qualifiers.
|
|
9
9
|
|
|
10
10
|
Classify each document by its semantic role: requirement/Change, implementation plan, Spec, Decision Record/ADR, context/navigation, operating instruction, or explanatory/user documentation. Then check:
|
|
@@ -12,21 +12,19 @@ Classify each document by its semantic role: requirement/Change, implementation
|
|
|
12
12
|
1. **Purpose:** identify the intended reader, available prior knowledge, and reading task. Missing framing matters when it causes a wrong decision or prevents that task, not merely when a preferred introduction is absent.
|
|
13
13
|
2. **Grounding:** verify authority and traceability of claims and decisions, including commands, defaults, conditions, limits, and failure outcomes. For shortened text, check lost negation, exceptions, permissions, quantities, units, certainty or necessary reasoning. Technical literals must not change under wording-only authority. Compare internal and cross-artifact meaning; polished prose can still make a false promise.
|
|
14
14
|
3. **Usability:** judge task usability separately from factual accuracy. Procedures need usable actions and success signals, contracts assessable conditions, and decisions reasons and tradeoffs. Against the intended reader's prior knowledge, check findability, prerequisites, definitions, causal and conditional links and visible unknowns without the author conversation. Retain mechanisms needed to reason, troubleshoot or judge guarantees; otherwise true internals may still leave the required action unclear. Preserve necessary teaching and local safety without demanding tutorials in references. Fixed order needs a correctness, safety or contract reason.
|
|
15
|
-
4. **Ownership:** check scope, duplicate authority, and choices disguised as settled decisions. An unresolved choice warrants an ambiguity Finding when it prevents the reviewed content from serving its declared task or safe use. Respect draft state and narrower review scope; intentionally deferred or out-of-scope choices are limits, not automatic defects.
|
|
16
|
-
5. **Maintenance:** check affected paths, links, anchors, terms and examples. For structural edits, follow reading paths and inspect moved content for omissions or broken ownership. Flag repeated examples or failure explanations only when they add no decision distinction. Preserve useful summaries, local safety and required evidence, including rare permission, serious-consequence and safe-recovery boundaries. An incident alone establishes no need for another constraint.
|
|
15
|
+
4. **Ownership:** check scope, duplicate authority, and choices disguised as settled decisions. An unresolved choice warrants an ambiguity Finding when it prevents the reviewed content from serving its declared task or safe use. Respect draft state and narrower review scope; intentionally deferred or out-of-scope choices are limits, not automatic defects.
|
|
16
|
+
5. **Maintenance:** check affected paths, links, anchors, terms and examples. For structural edits, follow reading paths and inspect moved content for omissions or broken ownership. Flag repeated examples or failure explanations only when they add no decision distinction. Preserve useful summaries, local safety and required evidence, including rare permission, serious-consequence and safe-recovery boundaries. An incident alone establishes no need for another constraint.
|
|
17
17
|
|
|
18
18
|
Check ownership by meaning, not file shape. CONTEXT owns canonical terms, domain relationships, and navigation; AGENTS owns operating rules; README owns introduction and usage. Specs own current contracts and necessary constraints, including design-reference tables when they define normative evidence. Change Spec owns only the delta and acceptance; Design owns the approach, Tasks the work, and Verify the evidence. Decision Records own rationale. Short summaries and references are valid; parallel full definitions are not.
|
|
19
19
|
|
|
20
|
-
Within fixed work-record scope,
|
|
20
|
+
Within fixed work-record scope, load the [current work record rules](../../rsp/references/work-model.md#current-work-records) when absent from context. Check Tasks, Verify and Focus against their respective rules, including current limits and stale conclusions. Existing text has no exemption. When judging blockers/readiness, compare actual impediments and the declaration with parsed state under the same work model. A claim of no blockers that parses as blocked is an inconsistency.
|
|
21
21
|
|
|
22
|
-
For closeout, inspect the final whole Change or archived artifact after needs-based writeback, not only added Verify lines or an earlier candidate. Check that any writeback uses verified results, necessary knowledge corrections are complete and the final Change reflects them without duplicating current specifications. Already-accurate knowledge needs no edit; required corrections cannot be waived. Whole-Change convergence removes stale operation permissions
|
|
22
|
+
For closeout, inspect the final whole Change or archived artifact after needs-based writeback, not only added Verify lines or an earlier candidate. Check that any writeback uses verified results, necessary knowledge corrections are complete and the final Change reflects them without duplicating current specifications. Already-accurate knowledge needs no edit; required corrections cannot be waived. Whole-Change convergence follows the current work record rules and removes stale operation permissions and response-only handoff/readiness blocks. Return substantive inconsistencies to their owner; prose cleanup cannot resolve them. Do not conceal unmet checks or limits, relabel reports or grant cleanup authority. CLI readiness cannot establish this semantic result.
|
|
23
23
|
|
|
24
|
-
|
|
24
|
+
Specialized Spec headings and omitted unnecessary scenarios are not defects; scaffold hints are optional writing aids, not requirements or evidence. Material promises, decisions, and limits must remain visible outside comments. Do not request extra sections, tests, or bulk historical rewrites merely for template conformity.
|
|
25
25
|
|
|
26
|
-
For
|
|
26
|
+
For document findings, identify the reader/task. For usability, identify what readers must infer and its effect on the task. An obscured action, scattered conditions, independent decisions crowded together or repetition can cause avoidable reconstruction or lookup. Cite the specific passage and effect; "too long" or "sounds generated" alone establishes no defect. Agreed expression rules are contracts; unagreed taste is not. Accept alternatives meeting the task and applicable contracts. Preserve helpful causal prose, navigation summaries and author voice. Keep contrasts resolving likely ambiguity; do not require one per rule or delete needed operations to shorten text. Length, list counts and format checks cannot replace judgment.
|
|
27
27
|
|
|
28
28
|
For example, an unexplained "the plan" can misdirect execution or force an avoidable search before the reader finds the right plan. A definition or prerequisite link that identifies it resolves that issue. Necessary prerequisite reading or a dense paragraph alone is no defect.
|
|
29
29
|
|
|
30
|
-
Reading difficulty is evidence to investigate
|
|
31
|
-
|
|
32
|
-
Anchor Findings to the smallest heading or claim. Do not apply code-style or test-coverage rules to semantic documents, auto-fix meaning, or rewrite prose for taste.
|
|
30
|
+
Reading difficulty is evidence to investigate. The five dimensions also guide writing, but Review uses this local reference and does not depend on Doc being installed. State whether reader comprehension was assessed locally or with actual independent evidence; never infer independence from a self-check or require a fixed number of rounds. Do not apply code-style or test-coverage rules to semantic documents.
|
|
@@ -4,7 +4,7 @@ description: Resolve bounded design questions read-only or clarify material choi
|
|
|
4
4
|
license: MIT
|
|
5
5
|
metadata:
|
|
6
6
|
author: oevery
|
|
7
|
-
version: "2026.
|
|
7
|
+
version: "2026.11.04.1"
|
|
8
8
|
---
|
|
9
9
|
|
|
10
10
|
# RSP Shape
|
|
@@ -23,6 +23,8 @@ Resolve needed RSP settings, including defaults and inheritance, through the sel
|
|
|
23
23
|
|
|
24
24
|
Read the request, authority, nearest project context, Change and Brief when selected, then only decisive evidence. A direct design question without a Change does not require one. A design question inside a tracked Change retains that WorkRef, but defaults to report-only; only explicit artifact authority permits updating its Design.
|
|
25
25
|
|
|
26
|
+
For structural or responsibility decisions, use [architecture knowledge](../rsp/references/architecture.md) to inspect any existing model and distinguish current facts from proposed design.
|
|
27
|
+
|
|
26
28
|
A request to shape, create, or refine a Change grants artifact authority unless no-edit; advice does not. If creation focuses it, preserve the exact prior focus and restore it immediately unless selection was requested. Shaping grants no other authority; preserve unrelated work.
|
|
27
29
|
|
|
28
30
|
Route concrete or ready work to implementation only when authorized and no design/planning question remains. An explicit read-only design request still ends with its report. Do not rewrite settled work for style or completeness.
|
|
@@ -15,15 +15,15 @@ Keep one observable outcome per Change:
|
|
|
15
15
|
- `Proposal`: outcome, scope, non-goals;
|
|
16
16
|
- `Spec`: settled contract delta and acceptance, referencing affected existing Specs rather than copying the baseline or inventing public promises;
|
|
17
17
|
- `Design`: approach, responsibility boundaries, and necessary tradeoffs, not repeated requirements or tasks;
|
|
18
|
-
- `Tasks`: checkable work, with prescribed order only where correctness, safety, or migration requires it;
|
|
19
|
-
- `Verify`:
|
|
18
|
+
- `Tasks`: current checkable work, with prescribed order only where correctness, safety, or migration requires it;
|
|
19
|
+
- `Verify`: settled acceptance evidence under the current work record rules below; do not introduce new public requirements;
|
|
20
20
|
- `Blockers`: dependencies and decisions, or `none`.
|
|
21
21
|
|
|
22
22
|
Keep the six sections; small changes need only proportionate content. Plain-prose HTML scaffold hints may remain or be removed; they guide writing, never supply requirements, evidence, or completed content. Resolve body placeholders and keep all material decisions visible outside comments.
|
|
23
23
|
|
|
24
24
|
Plan a test only when it protects observable behavior or a real boundary, adds distinct future confidence, avoids duplicate or implementation-detail coverage, and costs proportionately. Otherwise prefer smallest sufficient evidence and keep probes temporary.
|
|
25
25
|
|
|
26
|
-
|
|
26
|
+
Apply the [current work record rules](../../rsp/references/work-model.md#current-work-records) when shaping or revising tasks and verification; load them when absent from context. Replace obsolete plans within settled authority, keeping unmet obligations visible. A Change records current intent and progress during work, then the delivered result at closeout. Necessary stable knowledge stays with its owner; Focus only supplements missing recovery context.
|
|
27
27
|
|
|
28
28
|
Use domain language. Mention agents only as real product actors or constraints, not authors or execution narrators.
|
|
29
29
|
|
|
@@ -35,7 +35,7 @@ A Change is ready only when:
|
|
|
35
35
|
- product and mutation authority are settled;
|
|
36
36
|
- affected boundaries and material constraints are known;
|
|
37
37
|
- no hidden assumption can change implementation or acceptance;
|
|
38
|
-
- Tasks are
|
|
38
|
+
- Tasks are concrete enough for implementation to begin. Shaping does not execute them;
|
|
39
39
|
- Verify proves the result without process chronology; Blockers are truthful;
|
|
40
40
|
- one Change or a justified shallow Group is the smallest sufficient owner.
|
|
41
41
|
|
|
@@ -4,7 +4,7 @@ description: Collect and report one selected RSP WorkOwner's declared check evid
|
|
|
4
4
|
license: MIT
|
|
5
5
|
metadata:
|
|
6
6
|
author: oevery
|
|
7
|
-
version: "2026.10.
|
|
7
|
+
version: "2026.10.27.1"
|
|
8
8
|
---
|
|
9
9
|
|
|
10
10
|
# RSP Verify
|
|
@@ -15,14 +15,14 @@ Collect declared-check evidence for one existing RSP WorkOwner when requested or
|
|
|
15
15
|
|
|
16
16
|
## Require a declared boundary
|
|
17
17
|
|
|
18
|
-
Require one selected WorkOwner,
|
|
18
|
+
Require one selected WorkOwner, current target, declared check contract and execution authority. A comparison needs its comparison baseline; a current-target check does not need an old revision. An explicit reference wins; otherwise resolve one FocusSet candidate from user intent and status. Multiple markers are valid.
|
|
19
19
|
|
|
20
20
|
- **Change:** use its WorkRef and its `Verify` section for the declared boundary.
|
|
21
21
|
- **Group:** use its Group reference and named `Integration:` condition in the Brief's `Completion Conditions`; child evidence stays with each child WorkRef. An explicit request may supply an in-progress temporary boundary, which must be written back before Group closeout.
|
|
22
22
|
|
|
23
|
-
Stop for ambiguous owner,
|
|
23
|
+
Stop for ambiguous owner, target, required evidence or environment, a missing required comparison baseline, or a Group without a named boundary for the requested verification.
|
|
24
24
|
|
|
25
|
-
Read nearest
|
|
25
|
+
Read nearest instructions, selected Change/Brief, relevant contracts, blockers and the smallest production path needed to understand the check. Inspect the diff when the check concerns a change. This package needs no installed Core Skill. Do not invent checks from generic testability or replace an unexplained failure with a guessed assertion; return that symptom for read-only diagnosis.
|
|
26
26
|
|
|
27
27
|
## Preserve read-only authority
|
|
28
28
|
|
|
@@ -32,7 +32,9 @@ Use the caller's result contract when supplied, otherwise the direct result belo
|
|
|
32
32
|
|
|
33
33
|
## Collect the declared evidence
|
|
34
34
|
|
|
35
|
-
|
|
35
|
+
Bind evidence to the declared target. For comparison or reuse, establish the earlier target and relevant inputs. Reuse traceable passes while their tested scope and relevant content, dependencies, configuration, tools, inputs and environment still apply. Run missing, failed or invalidated named checks and any explicitly required execution, current-environment observations or independent execution. Corrections and integration need fresh evidence for affected inputs or an enlarged boundary; a phase change alone requires no rerun.
|
|
36
|
+
|
|
37
|
+
Create no backup, snapshot or digest merely to identify a routine check. Use them only for an actual recovery requirement or independent identity/integrity consumer. Exact Git delivery, trusted external digests and required recovery safeguards retain their own contracts.
|
|
36
38
|
|
|
37
39
|
Compare each check's actual targets, inputs, coverage and completion with its declaration. Retain commands, scope, sources and results; distinguish reused evidence from new execution. Invocation errors or incomplete observations alone prove no product defect; exit status alone establishes no acceptance.
|
|
38
40
|
|
|
@@ -52,14 +54,7 @@ Return exactly one canonical result from the collected evidence, with observed f
|
|
|
52
54
|
|
|
53
55
|
Report each check's outcome, including missing or unexecuted checks. Aggregation changes neither coverage nor stop conditions. Failure does not imply complete coverage; unavailable evidence does not erase observed failures.
|
|
54
56
|
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
- `evidence_delta: new | none`: whether the result changes the next diagnosis or correction.
|
|
58
|
-
- `boundary: unchanged | changed`: whether observed owner, paths, baseline, behavior, interface, scope and authority still match the declared boundary.
|
|
59
|
-
- WorkOwner and applicable Change WorkRef or Group `Integration:` condition; lane objective and effective authority.
|
|
60
|
-
- Named checks, comparison baseline, observed diff, decisive evidence with reused versus newly executed results, omissions and stop boundary.
|
|
61
|
-
|
|
62
|
-
Preserve exact machine values and all required result information. Present them concisely without repeated logs or evidence; a requested machine schema remains exact. Human-facing narration follows a supplied response-language contract; otherwise use explicit user instruction, then personal instructions, then conversation language.
|
|
57
|
+
Report the WorkOwner and applicable WorkRef or Group `Integration:` condition, target, named checks and outcomes, decisive evidence, omissions and any changed boundary. Distinguish reused from new execution; include a comparison baseline only for a comparative claim or reused evidence. Add machine fields only for an actual consumer or explicitly required schema, preserving its exact values. Do not repeat logs. Human-facing narration follows explicit user instruction, then personal instructions, then conversation language.
|
|
63
58
|
|
|
64
59
|
`pass` proves only declared checks, not semantic review, writeback, archive/commit eligibility, publication, deployment, approval or human acceptance. It grants no lifecycle, Git, external or acceptance authority.
|
|
65
60
|
|
|
@@ -67,6 +62,6 @@ Long-running verification remains active while its boundary and stop conditions
|
|
|
67
62
|
|
|
68
63
|
## Stop and return
|
|
69
64
|
|
|
70
|
-
Stop
|
|
65
|
+
Stop the affected verification if required evidence cannot be collected or observed, the required environment is unavailable, or the declared boundary changes. Use the caller's canonical stop reason only when supplied. For direct invocation, identify the missing input or changed boundary and the required next action; do not invent a completed verification result before the declared boundary is established.
|
|
71
66
|
|
|
72
67
|
On cancellation, wait until the command or owned process is observed stopped before beginning conflicting work. Return the result, decisive evidence, omissions and next action to the caller or user. Retry only with new evidence and safe replay inside the declared boundary.
|
|
@@ -1,43 +0,0 @@
|
|
|
1
|
-
# Artifact lenses
|
|
2
|
-
|
|
3
|
-
Choose what the reader must understand, do or judge, not a mandatory template. Preserve project formats and canonical headings; action-first wording is useful for procedures, not a requirement for every artifact.
|
|
4
|
-
|
|
5
|
-
For a document set, start with the reader's entry point and follow only the links needed for the task. Check that each handoff supplies its prerequisites and that summaries agree with the linked owner. An owner outside the authorized write set remains read-only; return its conflicting claim rather than expanding the edit boundary.
|
|
6
|
-
|
|
7
|
-
## README
|
|
8
|
-
|
|
9
|
-
- Help a newcomer find the starting point, prerequisites and first useful result with a recognizable success signal.
|
|
10
|
-
- State important exclusions before setup. Keep a useful overview and local prerequisites; link specialized detail with its reading purpose.
|
|
11
|
-
- Match the audience. A maintainer README need not become an installation tutorial.
|
|
12
|
-
|
|
13
|
-
## CONTEXT
|
|
14
|
-
|
|
15
|
-
- Help readers understand domain terms and relationships, then find the authoritative owner; do not duplicate that owner's full contract.
|
|
16
|
-
- Keep names, useful translations and navigation near each definition. Link one authoritative definition across root and local files.
|
|
17
|
-
- Explain different meanings at domain boundaries; do not silently rename them.
|
|
18
|
-
|
|
19
|
-
## Spec
|
|
20
|
-
|
|
21
|
-
- Enable a reader to judge compliance: state what a consumer can rely on, under which conditions, with scope, handoffs, failures and exceptions.
|
|
22
|
-
- Keep normative design references and protocol detail needed to establish compliance.
|
|
23
|
-
- Add a scenario only when it resolves ambiguity. Domain headings are valid; no fixed section set is required.
|
|
24
|
-
- Leave future promises in the Change and verification results with their work owner.
|
|
25
|
-
|
|
26
|
-
For example, “retries are safe” is incomplete when deduplication applies only to the same key. State that condition without inventing retention or failure guarantees.
|
|
27
|
-
|
|
28
|
-
## Guides and references
|
|
29
|
-
|
|
30
|
-
- Procedures: give prerequisites, execution context, bounded actions and success signals. Explain a likely failure when it helps the task; keep conditional branches near their triggers.
|
|
31
|
-
- Concepts and design explanations: preserve definitions, causal reasoning and tradeoffs needed to understand or decide; do not replace them with a terse action checklist.
|
|
32
|
-
- References: provide exact contracts and navigable details, not a forced tutorial.
|
|
33
|
-
- Label illustrative snippets; do not present invented output as observed execution. Preserve incoming links after heading changes.
|
|
34
|
-
- API/TSDoc comments: explain the consumer's real inputs, returns, failures and use. Compare examples and claims with their authoritative contract and implementation; report unresolved discrepancies rather than silently documenting whatever code does. Writing authority does not permit implementation changes, execution or dependency installation.
|
|
35
|
-
|
|
36
|
-
## Operating instructions and work records
|
|
37
|
-
|
|
38
|
-
- AGENTS: state applicable rules, commands, authority and checks. Changing rules requires explicit scope.
|
|
39
|
-
- Change: retain the six canonical sections and selected delta. Proposal gives the outcome; Spec gives acceptance; Design gives the approach; Tasks state checkable work and completion, with completed items describing the accomplished result; Verify records methods, candidate-bound conclusions, limits and necessary evidence references; Blockers uses the [shared work-model syntax](../../rsp/references/work-model.md) for current impediments.
|
|
40
|
-
- Work records: retain necessary plan/contract changes, outcomes, blockers and scoped verification. Keep routine narration in conversation and needed recovery state in Focus. Before archive, establish facts and justified knowledge writeback; reconcile all six sections with current outcomes, superseded plans and knowledge owners. Retain facts affecting the delivered contract, evidence applicability, important failure disposition or remaining limit. Link knowledge and evidence owners; preserve their detailed records and original verdicts. Correct stale authoritative contracts. Return substantive contradictions to their owner; prose cleanup cannot settle them.
|
|
41
|
-
- Decision Record: let readers understand why a choice was made, its alternatives and consequences, without redefining the contract.
|
|
42
|
-
- Audit/run records: retain required observations, actions and evidence. Remove irrelevant session narration, not the record's purpose.
|
|
43
|
-
- Skill packages: use Doc's Skill-authoring branch. Release communication uses Doc's conditional release method.
|
|
@@ -1,41 +0,0 @@
|
|
|
1
|
-
# Structural review
|
|
2
|
-
|
|
3
|
-
Load for requested structural discovery or a concrete structural risk within an already fixed review boundary. Select the smallest relevant set of lenses; do not turn them into a completion checklist.
|
|
4
|
-
|
|
5
|
-
## Candidate selection in a broad boundary
|
|
6
|
-
|
|
7
|
-
When the user supplies a broad repository boundary without a narrower lead, use repeated changes, recurring defects, or cross-owner correction history only to select the smallest candidate behavior chain. Churn is navigation evidence, not a Finding. Inspect the current owners and reachable path before applying any lens, and return `clean` when the selected chain has no concrete structural downside.
|
|
8
|
-
|
|
9
|
-
## Ownership and sources of truth
|
|
10
|
-
|
|
11
|
-
Trace where important state or data is created, validated, mutated, persisted, recovered, and consumed. Look for competing writers, duplicated authority, or a lifecycle whose transitions are split across owners that cannot enforce the same invariants. A cache, projection, generated artifact, or test fixture is not automatically a competing source of truth. Report only when a reachable path can diverge and the divergence has a concrete consequence.
|
|
12
|
-
|
|
13
|
-
## Module and dependency direction
|
|
14
|
-
|
|
15
|
-
Start from a direct consumer and list what it must know about the provider: policy, representation, ordering, failure behavior, configuration, and lifecycle. Look for dependency direction that forces an owner to understand another module's internals, or a boundary that leaks enough decisions to amplify ordinary changes. An import, large module, circular-looking name, or missing interface is not sufficient; establish actual caller burden or invalid-state exposure.
|
|
16
|
-
|
|
17
|
-
Before calling a layer redundant, trace what its removal would do. Eliminated complexity suggests pass-through; policy, invariants or failure knowledge dispersed into callers suggests useful locality. Thinness proves neither. Retain a Finding only with a reachable trigger and concrete impact in the live chain.
|
|
18
|
-
|
|
19
|
-
## Production-path reachability
|
|
20
|
-
|
|
21
|
-
Trace the smallest complete path from a real entry point to the behavior under examination. For an adapter, validator, normalizer, wrapper, or policy seam, name the direct production caller and its actual callee. Report a bypass only when the live consumer reaches a different path, and distinguish an unused seam from a seam used by another valid entry point. Do not treat API existence, registration, generated code, or an isolated passing test as proof of lifecycle integration.
|
|
22
|
-
|
|
23
|
-
## Change amplification and repeated knowledge
|
|
24
|
-
|
|
25
|
-
Use repository history only when it is available and necessary; otherwise trace one representative behavior across schemas, types, adapters, callers, configuration, tests, and documentation. Report amplification when one behavioral change requires independently maintained edits because ownership is fragmented. Similar text, generated output, deliberate boundary translation, and test fixtures are not duplication by themselves. Code reduction is never the objective.
|
|
26
|
-
|
|
27
|
-
## Verification mismatch
|
|
28
|
-
|
|
29
|
-
Map focused tests, assertions, or checks to the same production path and public behavior claimed by the implementation. Look for tests that exercise only an isolated seam while the live consumer bypasses it, or verification that cannot observe the relevant state transition or failure contract. Missing tests alone are not a structural finding; establish that existing evidence creates false confidence or leaves a material invariant unverified.
|
|
30
|
-
|
|
31
|
-
## Finding gate and ranking
|
|
32
|
-
|
|
33
|
-
Before retaining a finding, answer all of these from repository evidence:
|
|
34
|
-
|
|
35
|
-
1. What exact entry point, owner, caller, or artifact anchors it?
|
|
36
|
-
2. What reachable condition triggers the problem?
|
|
37
|
-
3. What realistic user, operator, data, or change-cost impact follows?
|
|
38
|
-
4. Which evidence confirms the full chain, and what remains unobserved?
|
|
39
|
-
5. What is the smallest correction or unresolved owner/design question?
|
|
40
|
-
|
|
41
|
-
Discard the finding when any of the first three answers is speculative. Rank retained findings by impact and confidence, not ease of cleanup. Use Review's severity and verdict semantics. Select only material supported findings; missing coverage stays explicit and does not justify an unqualified clean claim. Do not turn the report into an implementation, replacement design or automatic handoff.
|