pi-goal-list-loop-audit 0.26.2 → 0.26.3
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/extensions/reviewer.ts +14 -2
- package/package.json +1 -1
package/extensions/reviewer.ts
CHANGED
|
@@ -65,16 +65,28 @@ export interface Finding {
|
|
|
65
65
|
/** Leverage classification (contract item 5). Order matters: strategic
|
|
66
66
|
* and architectural win over bug/refactor — "should we rewrite this
|
|
67
67
|
* broken schema" is a decision, not a fix. */
|
|
68
|
+
// v0.26.3: the bare words "architectural"/"strategic" are REMOVED — they
|
|
69
|
+
// self-matched the reviewer's own vocabulary ("architectural-class",
|
|
70
|
+
// "architectural findings", the docs' mode matrix) and produced 3 junk
|
|
71
|
+
// findings on the 0.26.2 completion, observed live.
|
|
68
72
|
const CLASS_PATTERNS: Array<{ class: FindingClass; re: RegExp }> = [
|
|
69
|
-
{ class: "strategic", re: /\bshould we\b|\bdeprecat|ship this
|
|
70
|
-
{ class: "architectural", re: /\brewrite\b|new dependency|schema change
|
|
73
|
+
{ class: "strategic", re: /\bshould we\b|\bdeprecat|ship this\??/i },
|
|
74
|
+
{ class: "architectural", re: /\brewrite\b|new dependency|schema change|\bredesign\b/i },
|
|
71
75
|
{ class: "bug", re: /\bTODO\b|\bFIXME\b|\bbug\b|\bissue\b|regression|broken|\bfixme\b/i },
|
|
72
76
|
{ class: "refactor", re: /could be cleaner|consider refactoring|duplicat|refactor|left ?out|follow[\s-]?up|deferred|could be improved|improvement|enhancement|consider adding|would be nice|nice to have/i },
|
|
73
77
|
];
|
|
74
78
|
|
|
79
|
+
/** v0.26.3: lines that never carry findings — code, markdown tables, and
|
|
80
|
+
* the reviewer's own report/config vocabulary. Observed false positives
|
|
81
|
+
* from the 0.26.2 completion: a test("…architectural…") name, the
|
|
82
|
+
* INSTALL.md mode-matrix row, and ship-doc prose. */
|
|
83
|
+
const SKIP_LINE = /^\s*(test|it|describe|assert|expect)\s*\(|^\s*(const|let|var|function|import|export|require)\b|\{\s*\.\.\.\s*\}|,\s*\.\.\.$|^\s*\|/;
|
|
84
|
+
const REVIEWER_VOCAB = /architectural-class|bug-class|refactor-class|strategic-class|reviewer found|cascade step|\*\*Mode\*\*|problems\s*\/\s*(improvements|architectural)/i;
|
|
85
|
+
|
|
75
86
|
export function classifyFindingText(line: string): FindingClass | undefined {
|
|
76
87
|
const t = line.trim();
|
|
77
88
|
if (t.length < 8) return undefined;
|
|
89
|
+
if (SKIP_LINE.test(t) || REVIEWER_VOCAB.test(t)) return undefined;
|
|
78
90
|
for (const { class: cls, re } of CLASS_PATTERNS) {
|
|
79
91
|
if (re.test(t)) return cls;
|
|
80
92
|
}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "pi-goal-list-loop-audit",
|
|
3
|
-
"version": "0.26.
|
|
3
|
+
"version": "0.26.3",
|
|
4
4
|
"description": "Goal. Loop. Audit. Done. — a pi-coding-agent extension that supervises long-running work, with isolated auditor on each completion. Beat bamboozling by design: the auditor runs in a fresh session with no extensions, no skills, no editor — only the read tools needed to verify your goal.",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"author": "dracon",
|