@fro.bot/systematic 3.2.2 → 3.2.4
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/agents/document-review/adversarial-document-reviewer.md +5 -3
- package/agents/document-review/coherence-reviewer.md +6 -4
- package/agents/document-review/design-lens-reviewer.md +5 -3
- package/agents/document-review/feasibility-reviewer.md +5 -3
- package/agents/document-review/product-lens-reviewer.md +5 -3
- package/agents/document-review/scope-guardian-reviewer.md +5 -3
- package/agents/document-review/security-lens-reviewer.md +5 -3
- package/dist/pi.js +2840 -2415
- package/package.json +4 -4
- package/skills/document-review/SKILL.md +7 -6
- package/skills/document-review/references/findings-schema.json +5 -6
- package/skills/document-review/references/review-output-template.md +48 -34
- package/skills/document-review/references/subagent-template.md +11 -11
- package/skills/document-review/references/synthesis-and-presentation.md +3 -3
|
@@ -73,9 +73,11 @@ Probe whether the document considered the obvious alternatives and whether the c
|
|
|
73
73
|
|
|
74
74
|
## Confidence calibration
|
|
75
75
|
|
|
76
|
-
- **
|
|
77
|
-
- **
|
|
78
|
-
- **
|
|
76
|
+
- **0:** The challenge is a false positive or a pre-existing issue. Suppress it.
|
|
77
|
+
- **25:** The premise or assumption might be wrong, but the available document and codebase evidence cannot verify it. Suppress it.
|
|
78
|
+
- **50:** The challenge is verified, but it is an advisory or low-impact concern that does not materially threaten the decision. Return it as FYI only.
|
|
79
|
+
- **75:** You have double-checked the premise or decision against the document (and relevant codebase evidence where available), can construct a concrete failure scenario, and the consequence directly affects correctness in practice. This is actionable.
|
|
80
|
+
- **100:** The document directly confirms the premise failure or missing assumption, and the concrete failure scenario will occur frequently on the plan's normal path. Reserve this exceptional actionable anchor for direct evidence, not a persuasive counterargument; it is the only anchor eligible for a silent fix.
|
|
79
81
|
|
|
80
82
|
## What you don't flag
|
|
81
83
|
|
|
@@ -14,7 +14,7 @@ You are a technical editor reading for internal consistency. You don't evaluate
|
|
|
14
14
|
|
|
15
15
|
**Terminology drift** -- same concept called different names in different sections ("pipeline" / "workflow" / "process" for the same thing), or same term meaning different things in different places. The test is whether a reader could be confused, not whether the author used identical words every time.
|
|
16
16
|
|
|
17
|
-
**Structural issues** -- forward references to things never defined, sections that depend on context they don't establish, phased approaches where later phases depend on deliverables earlier phases don't mention. Also: requirements lists that span multiple distinct concerns without grouping headers. When requirements cover different topics (e.g., packaging, migration, contributor workflow), a flat list hinders comprehension for humans and agents. Flag with `autofix_class:
|
|
17
|
+
**Structural issues** -- forward references to things never defined, sections that depend on context they don't establish, phased approaches where later phases depend on deliverables earlier phases don't mention. Also: requirements lists that span multiple distinct concerns without grouping headers. When requirements cover different topics (e.g., packaging, migration, contributor workflow), a flat list hinders comprehension for humans and agents. Flag with `autofix_class: manual` when grouping requires a judgment call about the document's organization, keeping original R# IDs.
|
|
18
18
|
|
|
19
19
|
**Genuine ambiguity** -- statements two careful readers would interpret differently. Common sources: quantifiers without bounds, conditional logic without exhaustive cases, lists that might be exhaustive or illustrative, passive voice hiding responsibility, temporal ambiguity ("after the migration" -- starts? completes? verified?).
|
|
20
20
|
|
|
@@ -24,9 +24,11 @@ You are a technical editor reading for internal consistency. You don't evaluate
|
|
|
24
24
|
|
|
25
25
|
## Confidence calibration
|
|
26
26
|
|
|
27
|
-
- **
|
|
28
|
-
- **
|
|
29
|
-
- **
|
|
27
|
+
- **0:** The apparent inconsistency is a false positive or a pre-existing issue. Suppress it.
|
|
28
|
+
- **25:** The wording might be inconsistent, but the document does not provide enough evidence to verify that readers would diverge. Suppress it.
|
|
29
|
+
- **50:** The inconsistency is verified, but it is an advisory terminology or structural issue that is unlikely to affect implementation. Return it as FYI only.
|
|
30
|
+
- **75:** Two passages or a concrete reference comparison have been double-checked and would cause implementers to diverge or produce an incorrect interpretation in practice. This is actionable.
|
|
31
|
+
- **100:** Directly contradictory passages or a broken reference confirm an inconsistency that will recur frequently wherever the affected instruction is used. Reserve this exceptional anchor for direct textual evidence; it is the only anchor eligible for a silent fix.
|
|
30
32
|
|
|
31
33
|
## What you don't flag
|
|
32
34
|
|
|
@@ -35,9 +35,11 @@ Explain what's missing: the functional design thinking that makes the interface
|
|
|
35
35
|
|
|
36
36
|
## Confidence calibration
|
|
37
37
|
|
|
38
|
-
- **
|
|
39
|
-
- **
|
|
40
|
-
- **
|
|
38
|
+
- **0:** The design concern is a false positive or a pre-existing issue. Suppress it.
|
|
39
|
+
- **25:** A state, flow, or interaction might be missing, but the document does not let you verify the gap. Suppress it.
|
|
40
|
+
- **50:** The gap is verified, but it is advisory or low-impact and a reasonable implementation can proceed without resolving it. Return it as FYI only.
|
|
41
|
+
- **75:** You have double-checked a specific missing state, flow, or interaction against the document, and it will cause a practical implementation or user-flow failure. This is actionable.
|
|
42
|
+
- **100:** The document directly confirms a missing state or flow on a normal, frequently encountered interaction path, and the omission will repeatedly cause users to fail or implementers to block. Reserve this exceptional anchor for direct evidence; it is the only anchor eligible for a silent fix.
|
|
41
43
|
|
|
42
44
|
## What you don't flag
|
|
43
45
|
|
|
@@ -28,9 +28,11 @@ Apply each check only when relevant. Silence is only a finding when the gap woul
|
|
|
28
28
|
|
|
29
29
|
## Confidence calibration
|
|
30
30
|
|
|
31
|
-
- **
|
|
32
|
-
- **
|
|
33
|
-
- **
|
|
31
|
+
- **0:** The feasibility concern is a false positive or a pre-existing issue. Suppress it.
|
|
32
|
+
- **25:** The constraint or failure path might exist, but available document and codebase evidence cannot verify it. Suppress it.
|
|
33
|
+
- **50:** The constraint is verified, but it is an advisory or low-impact implementation concern that does not block the plan. Return it as FYI only.
|
|
34
|
+
- **75:** You have double-checked a concrete stack constraint, dependency, data-flow path, or migration condition and it will hit in practice, directly blocking correctness or implementation. This is actionable.
|
|
35
|
+
- **100:** Direct evidence from the stated stack, existing code, or an explicit plan constraint confirms that a normal path will fail frequently. Reserve this exceptional anchor for directly demonstrated, recurring incompatibility; it is the only anchor eligible for a silent fix.
|
|
34
36
|
|
|
35
37
|
## What you don't flag
|
|
36
38
|
|
|
@@ -59,9 +59,11 @@ If priority tiers exist: do assignments match stated goals? Are must-haves truly
|
|
|
59
59
|
|
|
60
60
|
## Confidence calibration
|
|
61
61
|
|
|
62
|
-
- **
|
|
63
|
-
- **
|
|
64
|
-
- **
|
|
62
|
+
- **0:** The product concern is a false positive or a pre-existing issue. Suppress it.
|
|
63
|
+
- **25:** The strategic concern might be real, but business context or evidence needed to verify it is absent. Suppress it.
|
|
64
|
+
- **50:** The misalignment is verified from the document, but it is an advisory, low-impact, or opportunity-cost observation. Return it as FYI only.
|
|
65
|
+
- **75:** You have double-checked the stated goal against the proposed work and the mismatch directly affects the likely outcome in practice. This is actionable. Qualitative strategic critiques top out here unless the document supplies direct contradiction or a hard boundary.
|
|
66
|
+
- **100:** Use this exceptional anchor only when the document directly contradicts its own stated goal or violates its own hard quantitative boundary, and the contradiction will occur frequently if followed. It is the only anchor eligible for a silent fix; do not use it for qualitative product judgment.
|
|
65
67
|
|
|
66
68
|
## What you don't flag
|
|
67
69
|
|
|
@@ -42,9 +42,11 @@ With AI-assisted implementation, the cost gap between shortcuts and complete sol
|
|
|
42
42
|
|
|
43
43
|
## Confidence calibration
|
|
44
44
|
|
|
45
|
-
- **
|
|
46
|
-
- **
|
|
47
|
-
- **
|
|
45
|
+
- **0:** The scope concern is a false positive or a pre-existing issue. Suppress it.
|
|
46
|
+
- **25:** The scope or complexity concern might be real, but the document does not provide enough evidence to verify it. Suppress it.
|
|
47
|
+
- **50:** The mismatch or complexity is verified, but it is advisory or low-impact and does not materially threaten delivery. Return it as FYI only.
|
|
48
|
+
- **75:** You have double-checked a stated goal against a scope item, abstraction, or priority boundary and the mismatch will directly affect right-sizing or delivery in practice. This is actionable. Qualitative scope critiques top out here unless the document supplies direct contradiction or a hard quantitative boundary.
|
|
49
|
+
- **100:** Use this exceptional anchor only when the document directly contradicts its own stated goal or violates its own hard quantitative boundary, and that contradiction will occur frequently if the plan is followed. It is the only anchor eligible for a silent fix; do not use it for qualitative complexity judgment.
|
|
48
50
|
|
|
49
51
|
## What you don't flag
|
|
50
52
|
|
|
@@ -26,9 +26,11 @@ Skip areas not relevant to the document's scope.
|
|
|
26
26
|
|
|
27
27
|
## Confidence calibration
|
|
28
28
|
|
|
29
|
-
- **
|
|
30
|
-
- **
|
|
31
|
-
- **
|
|
29
|
+
- **0:** The security concern is a false positive or a pre-existing issue. Suppress it.
|
|
30
|
+
- **25:** The threat might exist, but the plan and available context do not let you verify the exposure or missing control. Suppress it.
|
|
31
|
+
- **50:** The security gap is verified, but its impact is advisory or low and does not materially affect the described attack surface. Return it as FYI only.
|
|
32
|
+
- **75:** You have double-checked a concrete endpoint, trust boundary, input, secret, or data path against the plan and the missing control will directly affect security in practice. This is actionable.
|
|
33
|
+
- **100:** The plan directly confirms an unmitigated attack surface on a normal path, and the resulting exposure or exploit will occur frequently if implemented as written. Reserve this exceptional anchor for direct evidence and recurring impact; it is the only anchor eligible for a silent fix.
|
|
32
34
|
|
|
33
35
|
## What you don't flag
|
|
34
36
|
|