@mrciphersmith/keryx 0.3.123 → 0.3.125
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/README.md +4 -0
- package/dist/cli.js +1549 -556
- package/dist/core.js +85 -17
- package/package.json +1 -1
- package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +5 -5
- package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.detail.md +11 -11
- package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +25 -26
- package/src/gdskills/bundled/skills/review/review-premise/SKILL.md +2 -2
- package/src/gdskills/contracts/review-finding.schema.json +14 -1
|
@@ -96,7 +96,7 @@ judgement on it; the claim and the code read first let you see what the reviewer
|
|
|
96
96
|
|
|
97
97
|
`challenged` and `owner-decision` **hold** the finding: it does not count as an open
|
|
98
98
|
blocker or major, it is not sent to a fixer, and the verdict lists it as awaiting the
|
|
99
|
-
operator, who answers with `keryx
|
|
99
|
+
operator, who answers with the `review_decide_premise` card in keryx shell (the stronger path) or, elsewhere, to the host's own question tool (`uphold` or `dismiss`), which the caller records with `keryx review decide-premise --answered-by …` (a host question; never a terminal). You never
|
|
100
100
|
dismiss. `upheld` is the normal answer: a finding with a trigger, an outcome, no intent
|
|
101
101
|
signal and no practice source against it is upheld.
|
|
102
102
|
|
|
@@ -165,7 +165,7 @@ question, and the signals or sources behind it.
|
|
|
165
165
|
| Lowering a severity, holding a finding for the operator | YES | — |
|
|
166
166
|
| Whether a finding reproduces | NO | `review-verifier` |
|
|
167
167
|
| Adding a finding, raising a severity, editing one | **NO** | a domain reviewer in a new round |
|
|
168
|
-
| Dismissing a finding | NO | the operator: `keryx review decide-premise --answer dismiss` |
|
|
168
|
+
| Dismissing a finding | NO | the operator, by card or host question: `keryx review decide-premise --answer dismiss` |
|
|
169
169
|
|
|
170
170
|
## Red Flags
|
|
171
171
|
|
|
@@ -159,7 +159,20 @@
|
|
|
159
159
|
"quote": { "type": "string", "minLength": 1, "description": "The operator's own words: what they typed at the terminal, or the card text they said yes to." },
|
|
160
160
|
"at": { "type": "string", "minLength": 1 },
|
|
161
161
|
"actor": { "type": "string", "minLength": 1 },
|
|
162
|
-
"confirmedVia": { "type": "string", "enum": ["tui-card", "remote-card", "terminal"] }
|
|
162
|
+
"confirmedVia": { "type": "string", "enum": ["tui-card", "remote-card", "terminal", "host-question"] },
|
|
163
|
+
"hostQuestion": {
|
|
164
|
+
"type": "object",
|
|
165
|
+
"additionalProperties": false,
|
|
166
|
+
"required": ["source", "ref", "questionText", "answerText", "time"],
|
|
167
|
+
"description": "Set with confirmedVia host-question only: the question the host asked the operator and the answer that came back, as the agent reported them.",
|
|
168
|
+
"properties": {
|
|
169
|
+
"source": { "type": "string", "enum": ["claude-code", "codex", "keryx-shell", "helyx-poll", "other"] },
|
|
170
|
+
"ref": { "type": "string", "minLength": 1 },
|
|
171
|
+
"questionText": { "type": "string", "minLength": 1 },
|
|
172
|
+
"answerText": { "type": "string", "minLength": 1 },
|
|
173
|
+
"time": { "type": "string", "minLength": 1 }
|
|
174
|
+
}
|
|
175
|
+
}
|
|
163
176
|
}
|
|
164
177
|
}
|
|
165
178
|
},
|