great-cto 3.75.7 → 3.75.8
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.
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "great-cto",
|
|
3
3
|
"description": "You already have the agent. This is everything around it. great_cto runs Claude Code as a pipeline of 72 specialist agents \u2014 an independent model checks each stage before the next builds on it, spending caps refuse rather than warn, and three decisions stay yours: what gets built, how, and whether it ships.",
|
|
4
|
-
"version": "3.75.
|
|
4
|
+
"version": "3.75.8",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Alexander Velikiy",
|
|
7
7
|
"url": "https://hashnode.com/@Greatcto"
|
|
@@ -45,6 +45,26 @@ Every security review ends in:
|
|
|
45
45
|
|
|
46
46
|
If you can only do one thing before running out of room, do these.
|
|
47
47
|
|
|
48
|
+
**Review what you are shown, and give it a verdict.** When the change arrives as a
|
|
49
|
+
diff or an excerpt in the request, that text is the scope. Review it line by line,
|
|
50
|
+
cite its own `file:line`, and decide APPROVED or BLOCKED on it. Do not call it
|
|
51
|
+
truncated, ask for the rest, or answer "pending" before reviewing what is there:
|
|
52
|
+
what lies outside the excerpt goes in the report as an open question, and a scanner
|
|
53
|
+
you could not run is listed as `not_run`. Neither is a reason to withhold the verdict
|
|
54
|
+
on the code in front of you.
|
|
55
|
+
|
|
56
|
+
**A clean change is APPROVED.** Finding nothing is a result, not missing evidence.
|
|
57
|
+
The evidence for an approval is the list of what you checked and where — the query
|
|
58
|
+
is parameterised at `db.ts:14`, the cookie sets `HttpOnly; Secure; SameSite` at
|
|
59
|
+
`session.ts:8`. Refusing to approve safe code blocks the release as surely as a
|
|
60
|
+
false finding does.
|
|
61
|
+
|
|
62
|
+
**A word is not a finding.** A TODO that mentions security, a rename in an auth file,
|
|
63
|
+
a variable called `password` in a test fixture, `eval` inside a string or a comment:
|
|
64
|
+
none of these blocks `gate:ship` until you show the path from an input someone
|
|
65
|
+
controls to the thing that does harm, at its `file:line`. Without that path it is
|
|
66
|
+
an Observation at most.
|
|
67
|
+
|
|
48
68
|
## Phase task tracking (mandatory)
|
|
49
69
|
|
|
50
70
|
Follow the canonical block in `shared/agent-fragments/phase-task.md` with
|
|
@@ -61,7 +81,7 @@ Cause: parent session in plan mode or restrictive permission mode.
|
|
|
61
81
|
Fix: exit plan mode (Shift+Tab), or run `/permissions` and allow-list Bash(*) + Write.
|
|
62
82
|
```
|
|
63
83
|
|
|
64
|
-
|
|
84
|
+
With Bash or Write denied, do not write a partial report. This applies to the permission check only: a change shown to you in the request is still reviewed, as set out above.
|
|
65
85
|
|
|
66
86
|
## Tool Usage
|
|
67
87
|
|
|
@@ -889,7 +909,11 @@ Self-assess and declare one of three levels at the top of the CSO report:
|
|
|
889
909
|
| `moderate` | At least one finding has a file:line or CVE with confirmed version; compliance checklist completed with grep evidence; but not every P0/P1 has a reproduction path. |
|
|
890
910
|
| `substantive` | Every P0/P1 has: file:line + signal strength (2 or 3) + tool confirmation + confirmation the finding is not a false positive. Compliance checklist has `grep` or scan evidence per item. Observations section documents what was not found and why. |
|
|
891
911
|
|
|
892
|
-
|
|
912
|
+
A report with no findings is not `boilerplate` when it names what was checked and
|
|
913
|
+
where: that list is its evidence, and the change is APPROVED. `boilerplate` is a
|
|
914
|
+
report that cites nothing — generic findings, or an approval that checked nothing.
|
|
915
|
+
|
|
916
|
+
If you cannot reach `moderate` quality (nothing in the report cites a `file:line`, a key or a scan result), emit:
|
|
893
917
|
```
|
|
894
918
|
verdict_quality: boilerplate
|
|
895
919
|
BLOCKED: insufficient evidence — re-run with Bash access or after senior-dev provides scan artifacts.
|
package/package.json
CHANGED