@oevery/rsp 4.0.0 → 4.1.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-JO6NDJM4.mjs +360 -0
- package/dist/chunk-LGDCIQYL.mjs +725 -0
- package/dist/chunk-P3KO3GZW.mjs +1023 -0
- package/dist/chunk-U56XMIOO.mjs +66 -0
- package/dist/{chunk-HFSTLCCK.mjs → chunk-VDRXFKBP.mjs} +133 -1800
- package/dist/cli.mjs +275 -290
- package/dist/client-EHRWH5Q7.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-build.json +1 -0
- package/dist/viewer-service.mjs +551 -0
- package/package.json +21 -3
- package/skills/rsp/SKILL.md +7 -6
- package/skills/rsp/references/architecture.md +39 -0
- package/skills/rsp/references/durable-review.md +2 -2
- 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 +12 -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 +15 -11
- package/skills/rsp-review/SKILL.md +8 -8
- package/skills/rsp-review/references/code-review.md +11 -5
- package/skills/rsp-review/references/document-review.md +7 -9
- package/skills/rsp-shape/SKILL.md +3 -1
- package/skills/rsp-shape/references/planning.md +3 -3
- package/skills/rsp-verify/SKILL.md +8 -13
- package/skills/rsp-doc/references/artifact-lenses.md +0 -43
- package/skills/rsp-review/references/structural-review.md +0 -41
package/skills/rsp/SKILL.md
CHANGED
|
@@ -4,7 +4,7 @@ description: Coordinate authorized direct, Change or Group work; select capabili
|
|
|
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 Core
|
|
@@ -21,7 +21,7 @@ Before mutation, read nearest instructions and relevant CONTEXT; establish inten
|
|
|
21
21
|
|
|
22
22
|
Load the [work model](references/work-model.md) when tracked identities are missing from context; required tracking cannot be bypassed by relabelling work Direct. Use rsp status for needed owner, dependency or recovery facts, and JSON for exact dependency data. Required dependency checks and fresh archive readiness still apply.
|
|
23
23
|
|
|
24
|
-
Use the CLI invocation selected by the project AGENTS entry
|
|
24
|
+
Use the CLI invocation selected by the project AGENTS entry; `rsp` in these Skills denotes that selection. Resolve needed settings through the selected CLI's successful `rsp config --json` summary. Reuse it until relevant drift or recovery; raw YAML is diagnostic only. Projection failure blocks choices depending on unknown settings, while independent authorized work continues.
|
|
25
25
|
|
|
26
26
|
Select capabilities from their descriptions and the task contract. Executors load applicable methods and inspect sources; Core consults method details when needed for its own work or a concrete coordination question. Select the responsibility needed now:
|
|
27
27
|
|
|
@@ -32,7 +32,7 @@ Select capabilities from their descriptions and the task contract. Executors loa
|
|
|
32
32
|
- rsp-review: fixed-scope read-only judgment or explicitly bounded structural discovery. Supporting checks remain within that review.
|
|
33
33
|
- rsp-commit: separately authorized exact local delivery; required for Git execution, with no Core substitute if unavailable.
|
|
34
34
|
|
|
35
|
-
Continue authorized same-scope work without another continuation request. A phase, role or method change alone requires no tool call, worker, review or test run. Required review, independent execution and protected operations still apply. For ownerless release checks, establish the exact candidate, check contract, baseline
|
|
35
|
+
Continue authorized same-scope work without another continuation request. A phase, role or method change alone requires no tool call, worker, review or test run. Required review, independent execution and protected operations still apply. For ownerless release checks, establish the exact candidate, check contract and effects authority, plus a baseline for any comparison; invent no Change or Verify owner.
|
|
36
36
|
|
|
37
37
|
## Execute and reconcile evidence
|
|
38
38
|
|
|
@@ -40,7 +40,7 @@ Implement owns ordinary self-checks. Main-agent inspection can assess results, b
|
|
|
40
40
|
|
|
41
41
|
Reuse traceable observations, checks and reviews while their target, acceptance boundary and relevant inputs still apply. Changed code, dependencies, configuration, tools or environment invalidate affected evidence; role changes and delivery-scope confirmation alone do not. Include excluded worktree inputs when assessing a staged slice, and recognize behavior changes in normative Specs or executable Skills even when stored as Markdown.
|
|
42
42
|
|
|
43
|
-
Check paths, diff, outcomes, omissions and evidence applicability. Reuse executor checks when applicable. Independent semantic review may use implementer test evidence; required independent execution must run.
|
|
43
|
+
Check paths, diff, outcomes, omissions and evidence applicability. Reuse executor checks when applicable. Independent semantic review may use implementer test evidence; required independent execution must run. Preserve original evidence and verdicts with their existing owner; Change retains only applicable conclusions and material failure dispositions. Add no evidence ledger.
|
|
44
44
|
|
|
45
45
|
For required correction/re-review, load [review convergence](references/coordination-review.md). Batch compatible findings into one correction, run covering checks, then review finding resolution and new risks in the correction. Stop for changed scope/authority, unsafe replay, missing required capability, non-convergence or no new discriminating evidence. Required failures remain incomplete.
|
|
46
46
|
|
|
@@ -56,7 +56,7 @@ Load [interruption recovery](references/coordination-recovery.md) for pause/resu
|
|
|
56
56
|
|
|
57
57
|
## Maintain records and prepare delivery
|
|
58
58
|
|
|
59
|
-
During execution
|
|
59
|
+
During execution apply the [current work record rules](references/work-model.md#current-work-records), loading them when absent from context. Update affected Tasks/Verify directly when scope, contract, design and knowledge remain accurate; do not append attempts or postpone current-state corrections until archive. Use the same work model for Blockers syntax. Passing checks or recovery alone adds no whole-document rewrite. Focus supplies only missing recovery context, never acceptance or authority.
|
|
60
60
|
|
|
61
61
|
Ordinary completion and checkpoint commits leave Change open; a generic commit request does not establish final delivery. At established final delivery or an explicit close request, load [delivery closeout](references/closeout.md). workflow.closeout is ask (default), auto or manual; no setting grants Git, push, publication, deployment, approval or human acceptance authority.
|
|
62
62
|
|
|
@@ -66,7 +66,8 @@ Operate command-owned state through RSP commands. Persist only open and archived
|
|
|
66
66
|
|
|
67
67
|
Other conditional procedures:
|
|
68
68
|
|
|
69
|
-
- [
|
|
69
|
+
- [Architecture knowledge](references/architecture.md) when coordinating work on modeled structure or responsibilities.
|
|
70
|
+
- [Setup and repair](references/setup-repair.md) before initialization, managed-asset inspection, upgrade or repair.
|
|
70
71
|
- [Groups and dependencies](references/groups-dependencies.md) for grouped work.
|
|
71
72
|
- [Context migration](references/context-migration.md) when a root/relevant legacy CONTEXT-MAP.md is found or migration requested; preserve context, discovery alone grants no migration.
|
|
72
73
|
- [Conflict handling](references/conflict-handling.md) for an intersecting Git operation.
|
|
@@ -0,0 +1,39 @@
|
|
|
1
|
+
# Architecture knowledge
|
|
2
|
+
|
|
3
|
+
Use when a task concerns a project's structure, responsibilities, dependencies or
|
|
4
|
+
architecture model. Follow relevant CONTEXT navigation to `.rsp/architecture.json`
|
|
5
|
+
when present. The model maps current structure and relationships to documents;
|
|
6
|
+
Specs own contracts and decisions retain rationale. An absent model does not
|
|
7
|
+
require creation or block work.
|
|
8
|
+
|
|
9
|
+
## Read
|
|
10
|
+
|
|
11
|
+
Read only the relevant nodes, relations and bound documents, using the selected
|
|
12
|
+
project CLI's `rsp architecture --node <id> --json` or the model directly.
|
|
13
|
+
Use the current checkout/worktree's content. Treat the model as navigation and
|
|
14
|
+
check decisive claims against the owning code and contracts; it does not override
|
|
15
|
+
them or establish a mandatory Skill execution order.
|
|
16
|
+
|
|
17
|
+
## Update
|
|
18
|
+
|
|
19
|
+
When an authorized implementation changes modeled responsibilities, boundaries,
|
|
20
|
+
dependencies or document bindings, update the affected model entries with the
|
|
21
|
+
implemented facts. Preserve stable IDs for unchanged identities and keep detailed
|
|
22
|
+
contracts in their documents. Proposed structure stays in the Change Design until
|
|
23
|
+
implemented. Read-only Shape and Review report discrepancies without editing.
|
|
24
|
+
If a needed model correction is outside the authorized scope, return the gap to
|
|
25
|
+
the owner rather than silently expanding the task.
|
|
26
|
+
|
|
27
|
+
When authorized to add a model, register one link in root CONTEXT, reusing an
|
|
28
|
+
existing entry or creating minimal CONTEXT if absent. Installation or upgrade
|
|
29
|
+
alone neither creates the model nor rewrites context.
|
|
30
|
+
|
|
31
|
+
## Check
|
|
32
|
+
|
|
33
|
+
For affected model entries, implementation self-checks and Review compare the
|
|
34
|
+
represented responsibilities and relations with the changed code and contracts.
|
|
35
|
+
After model or bound-document changes, use `rsp doctor` for structural and
|
|
36
|
+
document/chapter-reference checks, reusing applicable passing evidence under the
|
|
37
|
+
ordinary verification rules. A pass does not establish semantic agreement with
|
|
38
|
+
implementation. Keep this within the task's existing checks and review scope;
|
|
39
|
+
unrelated tasks need no whole-model read, extra review or unconditional doctor run.
|
|
@@ -21,9 +21,9 @@ If inspection reports only `RSP project requires an update` without update autho
|
|
|
21
21
|
Apply this sequence only for established final delivery or preparation/audit of closure. Routine knowledge updates use the bounded writeback above.
|
|
22
22
|
|
|
23
23
|
1. **Verify outcome facts.** Establish the approved delivered result from current implementation, authoritative contracts and decisive evidence. Distinguish passed, failed, unverified and superseded conclusions. This establishes truth, not a preliminary full-document rewrite; plans and recovery notes alone cannot establish current facts.
|
|
24
|
-
2. **Distill Focus as needed.** Integrate only verified outcomes, evidence applicability or material limits missing from Change; no useful addition means no edit. Recovery state alone supplies no required acceptance evidence.
|
|
24
|
+
2. **Distill Focus as needed.** Integrate only verified outcomes, evidence applicability or material limits missing from Change; no useful addition means no edit. Resolve or safely transfer outstanding recovery obligations before discarding their notes. Recovery state alone supplies no required acceptance evidence.
|
|
25
25
|
3. **Write back knowledge as needed.** Using the verified result and Change delta, decide current facts and lasting rationale independently using the methods below. Update existing owners only when justified. No stable change means no edit; a delivered contract that leaves its authoritative Spec stale requires correction before closeout. Do not copy the whole Change or its verification history into knowledge documents.
|
|
26
|
-
4. **Converge the whole Change.** After necessary writeback, use `rsp-doc`
|
|
26
|
+
4. **Converge the whole Change.** After necessary writeback, use `rsp-doc`'s artifact method to reconcile all six sections under the [current work record rules](work-model.md#current-work-records). Retain the delivered delta, relevant design and result comparisons, applicable scoped evidence, material failure dispositions/limits and knowledge-update references; leave already-converged text unchanged. Remove obsolete plans and duplicate or misplaced content without moving history into a new report or Focus. Group integration conclusions stay in the Brief. Without Doc, perform the same bounded convergence locally without installation or replacing required Review.
|
|
27
27
|
5. **Check final consistency, then archive when authorized.** Compare the final Change and knowledge owners with the verified result; an earlier review of another revision or a passing readiness command proves no text quality. Reuse applicable implementation/test evidence, check new knowledge/document edits, and fill only missing or invalidated coverage. Document-only writeback and archive preparation do not themselves invalidate code checks. Complete separately required Review before the archive gate below. Fix prose defects in scope, but return material contract or implementation contradictions to the decision, implementation or verification owner. Unresolved Required failures still block closeout.
|
|
28
28
|
|
|
29
29
|
These are dependency constraints, not persisted phases or mandatory separate tool calls. Edit only what needs writeback; required updates cannot be waived. Resolve stale statements from verified outcomes. Preserve settled planning, unrelated style and original evidence/verdict owners; add no no-op reports. Author checks establish no independent review.
|
|
@@ -2,11 +2,15 @@
|
|
|
2
2
|
|
|
3
3
|
Load when tracked work needs a recovery snapshot, a Capsule needs inspection or replacement, or a portable continuation handoff is required. Same-session worker continuation alone uses [coordinated interruption and recovery](coordination-recovery.md), not this format. Short direct tasks need no new WorkRef or Capsule.
|
|
4
4
|
|
|
5
|
-
Core owns the selected WorkRef's
|
|
5
|
+
Core owns the selected WorkRef's optional recovery snapshot. At a pause, handoff or recovery-relevant checkpoint, create or replace it only when the existing snapshot is insufficient or stale. Keep only context unavailable from the Change, current workspace or existing evidence whose loss would make continuation unsafe or require costly repeated discovery. Ordinary same-session progress needs no update. Refer to Change items rather than maintain another task, verification or blocker list; a delivery blocker belongs in Change, with only missing operational detail here.
|
|
6
|
+
|
|
7
|
+
Retain the current recovery position, necessary evidence pointers, next action and resume conditions. Unfinished experiments, pending cleanup and operations with uncertain effects may matter; a ruled-out approach needs only its still-applicable conclusion when repeating it would be costly. Do not record command sequences, attempts, repair/test rounds or how the diagnosis evolved. Use `rsp focus <work-ref> --capsule-file <path|->` to replace the snapshot as facts change, not append history.
|
|
8
|
+
|
|
9
|
+
Remove notes when their obligations are discharged, obsolete or safely transferred to their existing owner. Do not clear unresolved effects or hard-to-reconstruct safety context merely to shorten the snapshot. An empty capsule file clears notes while retaining the focus marker; cleanup alone does not authorize unfocus, closure or deletion of evidence. Respect nearer mutation restrictions. For Group work use the selected child's Capsule, not a Group marker.
|
|
6
10
|
|
|
7
11
|
The existing v1 format is bounded to 4096 UTF-8 bytes: one leading `<!-- rsp-focus:v1 -->`, blank lines, exactly one non-empty single-line `Current`, `Evidence`, `Next`, and at most one non-empty single-line `Resume check`. No other non-empty lines are valid. Keep raw logs/diffs, runtime identifiers, host paths, credentials and execution history out. The marker path selects work; its notes grant no authority or acceptance and are never worker transport or controller state.
|
|
8
12
|
|
|
9
|
-
Before closeout, use the [durable-writeback sequence](durable-review.md)
|
|
13
|
+
Before closeout, use the [durable-writeback sequence](durable-review.md) to establish outcome facts and update their owners as needed. No useful Focus addition requires no edit; recovery obligations still need disposition before clearing the marker. Detailed evidence stays with its existing owner, never solely in the Capsule.
|
|
10
14
|
|
|
11
15
|
When accepted work remains, return a localized continuation with these semantic fields in order: `WorkOwner`, `Authority`, `Current state`, `Changed artifacts`, `Fresh verification`, `Blockers`, `Next action`. A Change WorkOwner uses its WorkRef; a Group WorkOwner uses its Group reference and direct child Changes. Preserve technical values; the continuation is not a second state store.
|
|
12
16
|
|
|
@@ -1,12 +1,15 @@
|
|
|
1
1
|
# Setup and repair
|
|
2
2
|
|
|
3
|
-
Load before initializing, inspecting or updating RSP-managed assets.
|
|
3
|
+
Load before initializing, inspecting or updating RSP-managed assets, including upgrades.
|
|
4
4
|
|
|
5
5
|
Use the runtime selected in AGENTS: an exact package pin in .rsp/version for
|
|
6
6
|
consumer projects, or this checkout's built CLI when .rsp/source explicitly
|
|
7
|
-
selects source mode. The two markers are mutually exclusive.
|
|
8
|
-
|
|
9
|
-
|
|
7
|
+
selects source mode. The two markers are mutually exclusive. Do not substitute a
|
|
8
|
+
published package for source mode because version strings match. An authorized
|
|
9
|
+
package upgrade deliberately selects its exact target CLI and runs its update for
|
|
10
|
+
the complete managed environment; source update keeps the local selection. Never
|
|
11
|
+
substitute @latest, a version range or an unverified global binary; package fetching
|
|
12
|
+
requires separate authority. Upgrade notices grant no migration authority.
|
|
10
13
|
|
|
11
14
|
- **Initialize:** rsp init [--agents-mode managed|print] [--with-project-setup] creates missing scaffolding and the complete core suite. Existing user files remain unchanged. Retry an incomplete init with the same target; repeated init cannot adopt a different pinned version. Retired Rules require update instead.
|
|
12
15
|
- **Update:** rsp update replaces the selected package's owned assets; optional --dry-run previews without writes. No separate installation or migration switches are needed.
|
|
@@ -33,8 +36,10 @@ Concurrent external edits and crash recovery are unsupported. No recovery entry
|
|
|
33
36
|
|
|
34
37
|
## Project knowledge
|
|
35
38
|
|
|
36
|
-
Keep architecture in .rsp/specs/design.md, contracts in
|
|
37
|
-
CONTEXT and scoped instructions outside the managed AGENTS block.
|
|
39
|
+
Keep architecture contracts in .rsp/specs/design.md, domain contracts in their Specs, navigation in
|
|
40
|
+
CONTEXT and scoped instructions outside the managed AGENTS block. For an existing
|
|
41
|
+
or explicitly requested architecture model, use [architecture knowledge](architecture.md)
|
|
42
|
+
for its relationship to contracts and its context entry. Use rsp specs to
|
|
38
43
|
query knowledge. A legacy context map selects [context migration](context-migration.md);
|
|
39
44
|
discovery alone grants no merge or deletion. Semantic writeback and archive decisions
|
|
40
45
|
remain outside deterministic setup. Missing required Skills stop dependent work;
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
# Shared work model
|
|
2
2
|
|
|
3
|
-
Read when selecting, reporting or operating a tracked RSP owner and
|
|
3
|
+
Read when selecting, reporting or operating a tracked RSP owner, or writing/reviewing its records, and the relevant definitions are not already available in context. Direct untracked work needs no RSP identity. These shared definitions require no Core execution.
|
|
4
4
|
|
|
5
5
|
## Work shapes
|
|
6
6
|
|
|
@@ -24,4 +24,14 @@ Identity, focus, readiness, dependency state and capability availability grant n
|
|
|
24
24
|
|
|
25
25
|
In a Change's Blockers, record only unresolved dependencies or decisions preventing required work. Write standalone `None` when none remain; keep non-blocking verification limits in Verify. Exact prerequisites use ``- requires `<work-ref>`: <reason>``. Other prose, including an explanation after `None`, remains an external blocker declaration.
|
|
26
26
|
|
|
27
|
-
Use only the selected owner's relevant artifacts and authoritative project context. Installed files are shared across the suite, but separate workers do not inherit one another's loaded instructions. Reuse applicable definitions in the same context; obtain this file before depending on missing definitions. If the shared resource is unavailable, report an incomplete suite and stop
|
|
27
|
+
Use only the selected owner's relevant artifacts and authoritative project context. Installed files are shared across the suite, but separate workers do not inherit one another's loaded instructions. Reuse applicable definitions in the same context; obtain this file before depending on missing definitions. If the shared resource is unavailable, report an incomplete suite and stop dependent actions rather than inventing a substitute or installing implicitly.
|
|
28
|
+
|
|
29
|
+
## Current work records
|
|
30
|
+
|
|
31
|
+
A Change retains six sections: Proposal, Spec, Design, Tasks, Verify and Blockers. During work it states the current authorized intent, approach and actual progress; at closeout it records the delivered outcome. Update affected content as facts change instead of appending rounds, dates or session summaries. Keep a date only when it matters to provenance or applicability. Record cleanup never authorizes dropping unmet scope or acceptance obligations.
|
|
32
|
+
|
|
33
|
+
- **Tasks:** keep current checkable work and completion. Replace, split or merge tasks within authorized scope; reopen or rewrite invalidated results. Routine attempts, fixes, retries and individual check runs are not new tasks. Completion proves neither verification nor independent or human acceptance.
|
|
34
|
+
- **Verify:** keep declared acceptance checks and their required/optional distinction, then retain only information needed to judge acceptance, evidence applicability or locate necessary evidence. Do not restate results already expressed by check items. Commands, counts, candidate/environment details and review steps belong here only when their omission would change those judgments or prevent that lookup; occurrence alone is no reason to retain them. Replace superseded conclusions only within their covered scope, preserve still-applicable reused evidence, and distinguish full review, correction-only re-review and human acceptance. Mark invalidated coverage incomplete. Keep shared evidence once and detailed execution/review records with their existing owner; no run history or debugging narrative.
|
|
35
|
+
- **Focus:** keep only additional context needed to resume safely or avoid costly repeated discovery; refer to Change state without maintaining a second task, verification or blocker list. The conditional [Focus method](focus-continuation.md) owns replacement and cleanup. Ordinary progress needs no snapshot.
|
|
36
|
+
|
|
37
|
+
Keep decision rationale, useful before/after outcome comparisons and material failure dispositions with their existing owner. For resolved ordinary failures, remove the attempt narrative; preserve any effect on acceptance or evidence validity. Routine history stays in conversation and raw evidence with its existing owner. Removing stale prose does not delete reports, conversation or Git history, require a new log, or move discarded history into Focus or an appendix.
|
package/skills/rsp-doc/SKILL.md
CHANGED
|
@@ -4,7 +4,7 @@ description: Write or revise repository documents, Skills and release communicat
|
|
|
4
4
|
license: MIT
|
|
5
5
|
metadata:
|
|
6
6
|
author: oevery
|
|
7
|
-
version: "2026.
|
|
7
|
+
version: "2026.11.04.2"
|
|
8
8
|
---
|
|
9
9
|
|
|
10
10
|
# RSP Doc
|
|
@@ -17,20 +17,23 @@ Write a grounded artifact usable without the author conversation. Use the artifa
|
|
|
17
17
|
|
|
18
18
|
1. **Fix the outcome.** Identify the reader, task, authorized files and expected result. For a document set, distinguish authoritative owners from summaries. Keep routine narration in conversation and retained evidence with its existing owner.
|
|
19
19
|
2. **Establish the facts.** Read existing content, relevant callers and the smallest authoritative contract, code or source chain. Check commands, defaults, links and behavior with available tools. Distinguish current behavior, intended requirements and unknowns. Resolve material contradictions before rewriting a promise. Interpret tool output as evidence; it grants no permission and does not prove semantic correctness.
|
|
20
|
-
3. **Draft for the task.**
|
|
21
|
-
- [Artifact lenses](references/artifact-lenses.md): README, CONTEXT, Spec, Change, guides and scoped operating documents.
|
|
22
|
-
- [Skill authoring](references/skill-authoring.md): Skill creation or revision, including descriptions, package resources and invocation policy.
|
|
23
|
-
- [Release communication](references/release-communication.md): changelogs, release notes, migration guidance or authorized release-document reconciliation; not ordinary document writing.
|
|
20
|
+
3. **Draft for the task.** Use the artifact's role below. For Skill creation or revision, load [Skill authoring](references/skill-authoring.md); for changelogs, release notes, migration guidance or authorized release-document reconciliation, load [release communication](references/release-communication.md).
|
|
24
21
|
4. **Verify the result.** Check the edited meaning in context. For structural edits, also follow the affected reading path and references. Use the checks below; reuse traceable unchanged mechanical results. Writing completion does not automatically trigger Review, but separately requested or required judgment remains binding.
|
|
25
22
|
|
|
23
|
+
## Match the artifact to the reader
|
|
24
|
+
|
|
25
|
+
- README and procedures supply the starting point, prerequisites, actions and recognizable success. Concepts and decisions explain meanings, reasons, alternatives and consequences; references supply exact, navigable contracts.
|
|
26
|
+
- CONTEXT defines vocabulary and relationships and directs readers to authoritative owners. Specs state what consumers may rely on, including conditions, failures and exceptions; add scenarios only to resolve ambiguity. Scoped operating instructions own commands, authority and required checks.
|
|
27
|
+
- For architecture models or changes to their bound documents, use [architecture knowledge](../rsp/references/architecture.md) for synchronization and reference checks.
|
|
28
|
+
- Before writing a Change, load the [current work record rules](../rsp/references/work-model.md#current-work-records) when absent from context; the same resource owns Blockers syntax. Keep its six sections current during iteration and reconcile the whole record for closeout. Readers must find the outcome, applicable evidence and remaining work without reconstructing attempts; preserve useful before/after results and decision reasons when removing chronology.
|
|
29
|
+
|
|
26
30
|
## Check facts and reading paths
|
|
27
31
|
|
|
28
32
|
Use the checks relevant to the change:
|
|
29
33
|
|
|
30
|
-
- Check whether readers can find information, understand conditions, act or judge compliance, and recognize unknowns.
|
|
31
34
|
- Check affected links, anchors, examples and metadata with existing tools. Execute examples only within safe authorized scope.
|
|
35
|
+
- Keep repository references portable: use project-root-relative file citations and document-relative Markdown links, following any existing machine-field path contract. Keep execution-root locations separate from durable file references; retain necessary external locations, original evidence and host-required display paths without copying local checkout prefixes into authored repository references.
|
|
32
36
|
- For linked artifacts, compare shared claims, terminology, defaults and handoffs against their authoritative owner. Correct authorized summaries and links without copying full definitions or changing unassigned owners. Report out-of-scope contradictions to their owner.
|
|
33
|
-
- Recheck facts, quantifiers, defaults, failures, permissions and terminology after editing. Report actual checks and remaining gaps.
|
|
34
37
|
- API comments and embedded guidance: compare claims with their contract and implementation without assuming either side correct. Keep ordinary inline comments locally accurate and useful: retain constraints, rationale or usage that nearby code does not make clear, and remove restatement that adds no information; no full writing workflow is needed. Compiler/tool annotations retain implementation ownership.
|
|
35
38
|
- Examples: label illustrative pseudocode or incomplete snippets; check copyable commands against real prerequisites, outcomes and safe use without inferring execution or installation authority.
|
|
36
39
|
|
|
@@ -47,12 +50,12 @@ For a failed command, inspect cause and actual effects before recovery. Change a
|
|
|
47
50
|
|
|
48
51
|
## Write precisely
|
|
49
52
|
|
|
50
|
-
-
|
|
53
|
+
- Explain unfamiliar concepts and mechanisms needed for the reader to reason, troubleshoot or assess guarantees. Do not force every artifact into an action list, tutorial or closing next step.
|
|
51
54
|
- State the actor, action and object directly. Prefer short, complete sentences when easier to follow. Separate independent actions; preserve causal and conditional links. Give each paragraph one useful purpose. Put prerequisites before use, conditions beside actions and exceptions beside their rules. Keep necessary local context; link branch-only detail with its reading condition and purpose.
|
|
52
55
|
- Number steps when their order matters; make each step a bounded action. Use peer bullets for independent items and connected prose for reasoning. Use tables for useful fixed mappings/comparisons, not branching Skill judgments.
|
|
53
|
-
- Remove
|
|
56
|
+
- Remove filler, repetition and irrelevant chronology. Keep author voice, decision-relevant reasoning and required audit evidence. Prefer concrete actions; use contrasts or examples when they resolve a real ambiguity. No fixed sentence length, list size or compression ratio defines quality.
|
|
54
57
|
- Preserve negation, conditions, exceptions, permissions, quantities, units and certainty. Merge failure explanations with the same decision effect. Retain permission, outcome, stop and safe-recovery conditions, including rare boundaries preventing unauthorized action, serious harm or unsafe replay. Keep incident details with their evidence owner. Do not turn a proposal or unknown into a verified fact. Shorter wording proves no behavioral equivalence.
|
|
55
|
-
- Wording edits preserve meaning,
|
|
58
|
+
- Wording edits preserve meaning, ownership, anchors and technical literals. Structural edits may merge, move or reorder content within scope, checking completeness and references. When the request authorizes simplifying instructions, remove an unnecessary obligation rather than merely restating it; verify the remaining behavior and safety. Wording-only authority does not permit that contract change.
|
|
56
59
|
- Use ordinary words for conjunctions. Preserve technical syntax; reserve | for closed alternatives, / for established pairs, → for defined transitions and : for label/value boundaries.
|
|
57
60
|
|
|
58
61
|
## Preserve ownership
|
|
@@ -6,7 +6,7 @@ Use for creating or revising a Skill. Apply Doc's language, expression and autho
|
|
|
6
6
|
|
|
7
7
|
- For creation, resolve the goal, inputs and observable result before choosing structure.
|
|
8
8
|
- Honor an explicit destination; otherwise use project instructions and host constraints. Check for existing content before creating files.
|
|
9
|
-
- For revision, inspect the authored package and its callers.
|
|
9
|
+
- For revision, inspect the authored package and its callers. State the revision's purpose and supporting evidence; preserve unrelated adaptations.
|
|
10
10
|
- Start with SKILL.md. Use a host initializer when useful; add resources only for a real task need.
|
|
11
11
|
- Preserve existing packages and global configuration. Creation is not installation or publication.
|
|
12
12
|
|
|
@@ -20,7 +20,7 @@ Use for creating or revising a Skill. Apply Doc's language, expression and autho
|
|
|
20
20
|
## Make execution concise
|
|
21
21
|
|
|
22
22
|
- Organize the ordinary task around its goal, necessary inputs, authorized action, completion evidence and real stops. Choose headings for the task, not a universal template. Continue ordinary authorized checks. Prescribe order only for correctness, safety or contract dependencies.
|
|
23
|
-
-
|
|
23
|
+
- Revise the existing owner before adding a rule for the same decision. Merge repeated conditions and remove instructions no longer needed within the authorized contract delta. An incident warrants a new constraint only for a reusable gap, not noncompliance or tool/fixture failure alone.
|
|
24
24
|
- Keep shared guidance and necessary local safety context visible. Link substantial branch-only knowledge with its reading condition and purpose; ordinary tasks need no unrelated recovery or delivery detail. A short Skill needs no router or extra files. Split or merge resources only when it improves the reading path, not when it merely moves mandatory text. Honor the declared distribution boundary. RSP ships as a suite: shared concepts may reference Core resources directly; independent Skills keep their own resource closure. Never depend on a maintainer repository Spec.
|
|
25
25
|
- Retain generic teaching or old model-compensation instructions only for a current task-specific need. A newer model label alone proves neither redundancy nor safe removal of a constraint.
|
|
26
26
|
- Reuse tools for deterministic work; the Skill owns method selection and interpretation within its authority. Preserve named mandatory commands, provenance, independence and protected operations when replacing a tool-only method. Add a script only when repetition or fragility warrants it, and run changed helpers safely. Check incoming links before moving resources.
|
|
@@ -4,7 +4,7 @@ description: Diagnose unexplained symptoms read-only or implement authorized bou
|
|
|
4
4
|
license: MIT
|
|
5
5
|
metadata:
|
|
6
6
|
author: oevery
|
|
7
|
-
version: "2026.
|
|
7
|
+
version: "2026.11.04.3"
|
|
8
8
|
---
|
|
9
9
|
|
|
10
10
|
# RSP Implement
|
|
@@ -22,12 +22,18 @@ Query RSP settings only for a decision depending on an unresolved value. Use the
|
|
|
22
22
|
Mutation requires one of:
|
|
23
23
|
|
|
24
24
|
- A bounded direct task with explicit scope and acceptance; no WorkRef or RSP writeback.
|
|
25
|
-
-
|
|
25
|
+
- A ready Change selected by explicit WorkRef or resolved from the FocusSet, with settled acceptance and exact product authority. Multiple markers are candidates; a Group Brief is context, not executable work.
|
|
26
26
|
|
|
27
27
|
Diagnosis-only investigates a bounded symptom read-only without a ready mutation owner or WorkRef. A suspected cause grants no mutation. For an authorized fix, diagnose an unexplained cause first; after confirmation continue the same request only with settled owner, scope and mutation authority.
|
|
28
28
|
|
|
29
29
|
Inspect applicable instructions, the selected Change/Brief when tracked, relevant Specs and decisions, and the owning code and tests. Use supplied paths and discover missing context as needed; a dispatch summary does not replace source inspection. Operate only in the execution location supplied by the host.
|
|
30
30
|
|
|
31
|
+
For optional context discovery, distinguish no matches from execution errors: no matches should not block independent subsequent reads, while missing required inputs and actual execution errors still need handling.
|
|
32
|
+
|
|
33
|
+
Within the confirmed checkout/worktree, prefer paths relative to the command or tool's explicit working directory for repository reads, searches and edits. Reuse the known execution root; do not switch worktrees to make a path resolve. Use absolute paths when the tool requires them, accessing files outside that root, or explicitly locating another authorized root; a path alone does not establish checkout identity or revision.
|
|
34
|
+
|
|
35
|
+
When work affects modeled structure, responsibilities, dependencies or bindings, use [architecture knowledge](../rsp/references/architecture.md) for reading, synchronization and own checks.
|
|
36
|
+
|
|
31
37
|
Select the method from the request and evidence:
|
|
32
38
|
|
|
33
39
|
- **Diagnosis:** for an unexplained symptom or diagnosis-only request, load [diagnosis](references/diagnosis.md) before any fix. Diagnosis-only remains read-only and ends with cause evidence or an unresolved discriminating check.
|
|
@@ -45,19 +51,17 @@ Git delivery, publication, deployment, approval, and out-of-scope deletion requi
|
|
|
45
51
|
|
|
46
52
|
## Admit boundaries and permanent tests
|
|
47
53
|
|
|
48
|
-
Inspect existing owners and callers
|
|
54
|
+
Inspect existing owners and callers. Prefer, in order, no addition when the outcome is already met, repository reuse, platform or standard-library facilities, existing dependencies, then the smallest complete implementation. Keep rules with their invariants; do not force unrelated responsibilities through flags or speculative interfaces.
|
|
49
55
|
|
|
50
56
|
Hide internal representation and coordination behind the necessary caller contract, including ordering and failures. Judge decomposition by what callers and maintainers must understand, not file size or layer count.
|
|
51
57
|
|
|
52
|
-
Add
|
|
58
|
+
Add defensive logic or a new boundary only for an explicit requirement or an evidenced producer, production consumer, trust/lifecycle transition and material consequence. Typed same-process values and imagined or test-only consumers do not establish that need. Backups, snapshots and digests likewise need a real recovery or independent identity/integrity consumer; do not create them as routine preflight. Preserve required recovery safeguards and independently trusted integrity checks.
|
|
59
|
+
|
|
60
|
+
Test stable public behavior with real internal collaborators by default. Substitute external or nondeterministic boundaries as needed. Focused unit tests remain useful for meaningful algorithms, parsers and critical boundaries; choose evidence by risk, not a per-function/file/branch allocation or unit/integration ratio.
|
|
53
61
|
|
|
54
|
-
|
|
62
|
+
Reuse sufficient existing evidence before adding a permanent test. A new test needs a distinct plausible regression, observable consequence and evidence gap worth its maintenance. Exercise the consumer outcome through the relevant production path; an already-covered forwarding hop adds no confidence.
|
|
55
63
|
|
|
56
|
-
|
|
57
|
-
- **Expectations and setup:** derive outcomes from the agreed contract or an independent example, not the tested computation. If fixture construction, platform behavior, fault injection or scheduling leaves the target condition uncertain, inspect the arranged state or use a targeted probe. This requires no precondition assertion or mutation run for every test.
|
|
58
|
-
- **Real consequence:** use real collaborators when fast, deterministic and practical; otherwise substitute the necessary boundary while preserving the relevant behavior. Exercise consumer-owned behavior through covered downstream adapters or forwarding hops. A nearest-spy, emitted-message or forwarding assertion covering only an already-covered hop adds insufficient evidence, even in an existing test file.
|
|
59
|
-
- **Assertions:** one final-message assertion can suffice for the agreed consumer outcome. Pin error details or interaction counts/order only when the contract needs them; otherwise assert outcomes and effects, including rejection.
|
|
60
|
-
- **Maintenance:** keep scenario-defining inputs and assertions visible; share incidental setup for clarity rather than repetition alone. Keep nearby tests protecting independent consequences, but do not allocate tests per touched file, wrapper, hop, branch or constant.
|
|
64
|
+
Derive expectations from the contract or independent examples. Assert outcomes and effects, including rejection; pin details, counts or order only when contract-relevant. Private structure, source wording and constants compared with themselves are not contracts; public defaults can be. When setup leaves the target condition uncertain, inspect it or use a targeted probe rather than adding ritual preconditions. Keep scenario-defining inputs and assertions visible and share incidental setup when clearer.
|
|
61
65
|
|
|
62
66
|
## Continue within the evidence boundary
|
|
63
67
|
|
|
@@ -67,7 +71,7 @@ Stop for a repeated failure without new discriminating evidence, non-convergence
|
|
|
67
71
|
|
|
68
72
|
## Implement and verify
|
|
69
73
|
|
|
70
|
-
Implement the smallest complete slice.
|
|
74
|
+
Implement the smallest complete slice. For authorized Change updates, load the [current work record rules](../rsp/references/work-model.md#current-work-records) when absent from context. Update affected work and evidence as facts change; reopen invalidated results rather than append fix/retry tasks or check history. Plan or contract changes retain their decision authority. Direct work reports outcomes without RSP artifact mutation.
|
|
71
75
|
|
|
72
76
|
Use `rsp-doc` for document/Skill writing and whole-Change convergence within existing authority. Local wording fixes and mechanical status/evidence updates needing no whole-record reconciliation remain direct. Without Doc, write grounded guidance and check affected examples/links locally. Author self-checks never certify independent Review.
|
|
73
77
|
|
|
@@ -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
|
-
After fixing an explicit structural-discovery scope, load [structural review](references/structural-review.md) for ownership, dependency, production-path and verification lenses; no diff is required. Within fixed-change review, use those lenses only for a concrete structural risk in the same boundary; do not expand into a repository audit. Return evidenced findings and the smallest correction or unresolved design question, not a replacement design or implementation. Keep the report and verdict semantics below.
|
|
33
|
-
|
|
34
32
|
Select methods from the judged content and its risks. Supporting references inform the judgment without becoming additional review targets. Respect narrower requested scope and reuse applicable instructions already in context.
|
|
35
33
|
|
|
36
|
-
- **[Code](references/code-review.md):** behavior, contracts, safety and failures of
|
|
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
|
|
|
@@ -43,6 +41,8 @@ When fixed reviewed artifacts include changelogs, release notes, migration guide
|
|
|
43
41
|
|
|
44
42
|
Inspect checkout state and the selected diff or structural boundary, then authority, then the smallest behavior chain and evidence needed for the question. Match checks to each perspective, actual content and changed risk. 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.
|
|
45
43
|
|
|
44
|
+
When the reviewed scope affects modeled structure, responsibilities, dependencies or bindings, apply [architecture knowledge](../rsp/references/architecture.md) within that scope.
|
|
45
|
+
|
|
46
46
|
Reuse traceable test evidence while its tested scope and relevant code, dependencies, configuration, tools, inputs and environment still apply. Run authorized checks only for gaps, invalidated results, changed risk or explicitly required execution. Distinguish reused evidence from new observations.
|
|
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.
|
|
@@ -58,16 +58,16 @@ Responses follow explicit language instruction, personal instructions, then conv
|
|
|
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. Two perspectives require neither two reports nor two reviewers.
|
|
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 that Tasks reflects current work, each Verify detail serves acceptance, applicability or necessary evidence lookup without repeating check results, and Focus contains only additional recovery context without a second state list or execution timeline. A record can still contain unnecessary execution detail without chronological headings. Preserve useful before/after comparisons, material failure dispositions and distinct review/acceptance boundaries. 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
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, attempt/session chronology and response-only handoff/readiness blocks while retaining the contract delta, material failures, version-bound results and evidence references. Return substantive inconsistencies to their owner; prose cleanup cannot resolve them. Do not hide failures, 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`: required/optional checks proving settled acceptance, not adding new public requirements; then applicable scoped results and gaps;
|
|
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
|
|