@only1btayy/g2w 1.0.23 → 1.0.25
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 +1 -1
- package/package.json +1 -1
- package/skills/bring2life.md +62 -0
- package/skills/ready2save.md +19 -8
- package/skills/the-challenger.md +3 -1
- package/skills/the-inspector.md +6 -4
- package/skills/the-leader.md +9 -1
package/README.md
CHANGED
|
@@ -118,7 +118,7 @@ Each Foundation agent is also a direct slash command. The normal flow runs throu
|
|
|
118
118
|
| `/g2w:the-challenger` | You wrote your own plan outside G2W and want it stress-tested before building |
|
|
119
119
|
| `/g2w:the-builder` | Plan is already locked — skip straight to building |
|
|
120
120
|
| `/g2w:the-inspector` | Code is already written — just verify it against a plan |
|
|
121
|
-
| `/g2w:the-leader` | Kick off the full pipeline
|
|
121
|
+
| `/g2w:the-leader` | Kick off the full pipeline — includes automatic code audit before Challenger reviews |
|
|
122
122
|
|
|
123
123
|
These are escape hatches for engineers who don't need the ceremony. G2W has no ceiling.
|
|
124
124
|
|
package/package.json
CHANGED
package/skills/bring2life.md
CHANGED
|
@@ -99,6 +99,68 @@ Generate all 11:
|
|
|
99
99
|
- `CURRENT.md`
|
|
100
100
|
- `PLAN.md` (leave empty — written by The Visionary when a task begins)
|
|
101
101
|
|
|
102
|
+
## Phase 2b — Code Audit
|
|
103
|
+
|
|
104
|
+
After generating draft docs, run a deep audit of the actual code. Do not wait for the user to ask — this happens automatically. The goal is to show the user the real health of their codebase before they start building on top of it.
|
|
105
|
+
|
|
106
|
+
Scan for:
|
|
107
|
+
|
|
108
|
+
**Bugs & crashes:**
|
|
109
|
+
- Race conditions, thread safety issues
|
|
110
|
+
- Null/undefined access paths
|
|
111
|
+
- Logic errors (duplicate checks, unreachable code, off-by-one)
|
|
112
|
+
- State that can get out of sync
|
|
113
|
+
|
|
114
|
+
**Dead code:**
|
|
115
|
+
- Files with zero references
|
|
116
|
+
- Functions/classes never called
|
|
117
|
+
- Directories that aren't compiled or imported
|
|
118
|
+
- For each: list the file, size, and verdict (delete / flag / verify first)
|
|
119
|
+
|
|
120
|
+
**Anti-patterns:**
|
|
121
|
+
- Copy-pasted logic that should be shared
|
|
122
|
+
- Hardcoded values that should be configurable
|
|
123
|
+
- Missing error handling at system boundaries
|
|
124
|
+
- Overly complex functions that do too many things
|
|
125
|
+
|
|
126
|
+
**Security:**
|
|
127
|
+
- Exposed secrets, API keys, credentials
|
|
128
|
+
- Unvalidated user input
|
|
129
|
+
- Unsafe file operations
|
|
130
|
+
|
|
131
|
+
**Flags worth checking:**
|
|
132
|
+
- Things that work in dev but would break in production
|
|
133
|
+
- Assumptions about paths, environments, or dependencies
|
|
134
|
+
|
|
135
|
+
Write everything to `ERRORS.md` in the project's G2W doc folder. Use this format:
|
|
136
|
+
|
|
137
|
+
```
|
|
138
|
+
# Errors & Issues — [project-name]
|
|
139
|
+
Generated by bring2life: [date]
|
|
140
|
+
|
|
141
|
+
## Critical Bugs
|
|
142
|
+
### ERR-XX — [short title]
|
|
143
|
+
**File:** `path/to/file.ext:line-range`
|
|
144
|
+
**What's wrong:** [explain what the code actually does vs what it should do]
|
|
145
|
+
**The fix:** [specific fix, not vague advice]
|
|
146
|
+
|
|
147
|
+
## Dead Code
|
|
148
|
+
| File | Size | Verdict |
|
|
149
|
+
|------|------|---------|
|
|
150
|
+
| `file.ext` | XXkb | Delete — [reason] |
|
|
151
|
+
|
|
152
|
+
## Anti-Patterns
|
|
153
|
+
[list with file references]
|
|
154
|
+
|
|
155
|
+
## Security
|
|
156
|
+
[list with file references]
|
|
157
|
+
|
|
158
|
+
## Flags
|
|
159
|
+
[things that need human verification before acting on]
|
|
160
|
+
```
|
|
161
|
+
|
|
162
|
+
Be specific. File paths, line numbers, what's actually broken and why. No vague "consider improving" — either it's a real issue or don't list it.
|
|
163
|
+
|
|
102
164
|
## Phase 3 — Gap Interview
|
|
103
165
|
|
|
104
166
|
After generating drafts, ask the questions you couldn't answer from the code alone. Ask them conversationally — no more than 3 at a time. Cover:
|
package/skills/ready2save.md
CHANGED
|
@@ -9,14 +9,19 @@ You are closing out this session. Leave the project in a state where any future
|
|
|
9
9
|
|
|
10
10
|
## Steps
|
|
11
11
|
|
|
12
|
-
1. **
|
|
12
|
+
1. **Capture decisions** — Review the conversation history and extract:
|
|
13
|
+
- What was built or changed
|
|
14
|
+
- Key decisions made and the reasoning behind them
|
|
15
|
+
- Any "don't do this again" lessons learned
|
|
13
16
|
|
|
14
|
-
|
|
15
|
-
> "What key decisions did we make this session that future-me needs to know? (reasoning behind choices, not just what was done)"
|
|
17
|
+
Do NOT ask the user to recall this — you have the full conversation. Only ask if something is genuinely ambiguous.
|
|
16
18
|
|
|
17
|
-
|
|
19
|
+
2. **Confirm the active project** — List all folders inside `projects/` and ask:
|
|
20
|
+
> "Which project are we saving? (pick one)"
|
|
18
21
|
|
|
19
|
-
|
|
22
|
+
Wait for the user to confirm. Do not assume based on what was discussed — the user decides.
|
|
23
|
+
|
|
24
|
+
3. **Update `.g2w/CURRENT.md`** in the confirmed project folder with exactly three sections:
|
|
20
25
|
```
|
|
21
26
|
## Last Completed
|
|
22
27
|
[What was finished and verified this session — be specific]
|
|
@@ -28,13 +33,19 @@ You are closing out this session. Leave the project in a state where any future
|
|
|
28
33
|
[The single most important next task]
|
|
29
34
|
```
|
|
30
35
|
|
|
31
|
-
|
|
36
|
+
Then write (or overwrite) `.g2w/CURRENT.md` in the **`Claudes Brain` root** with a single line:
|
|
37
|
+
```
|
|
38
|
+
active: projects/[confirmed-project-folder-name]
|
|
39
|
+
```
|
|
40
|
+
This is the pointer `back2it` uses to find the right project next session.
|
|
41
|
+
|
|
42
|
+
4. **Write a handoff note** at the bottom of `.g2w/CURRENT.md` under `## Session Notes — [date]`:
|
|
32
43
|
- What was built
|
|
33
44
|
- Key decisions made AND the reasoning behind them (the WHY matters more than the WHAT)
|
|
34
45
|
- Any gotchas or "don't do this again" lessons
|
|
35
46
|
- What to read first next session
|
|
36
47
|
|
|
37
|
-
4b. **Mine into MemPalace (if installed):** Run `/
|
|
48
|
+
4b. **Mine into MemPalace (if installed):** Run `/mempalace:mine` on the project's `.g2w/CURRENT.md` to index this session's decisions into MemPalace. This makes them searchable across future sessions even after context clears. If MemPalace isn't installed, skip this step silently.
|
|
38
49
|
|
|
39
50
|
5. **Commit** everything that was verified this session:
|
|
40
51
|
```
|
|
@@ -54,5 +65,5 @@ You are closing out this session. Leave the project in a state where any future
|
|
|
54
65
|
|
|
55
66
|
- Never commit unverified work — if something wasn't tested, say so and don't include it
|
|
56
67
|
- The handoff note must capture WHY decisions were made, not just what was done
|
|
57
|
-
-
|
|
68
|
+
- Extract decisions from the conversation — never make the user recall work you already witnessed
|
|
58
69
|
- Do not clear context or close anything — that's the user's call
|
package/skills/the-challenger.md
CHANGED
|
@@ -42,7 +42,9 @@ Attack every dimension:
|
|
|
42
42
|
|
|
43
43
|
1. **Read `PLAN.md`** from `~/.g2w/projects/[active-project]/`
|
|
44
44
|
|
|
45
|
-
1b. **
|
|
45
|
+
1b. **Read `ERRORS.md`** from `~/.g2w/projects/[active-project]/` — this is the code audit generated by The Leader. Use it as context: if the plan touches code with known bugs, flag it. If the plan ignores critical issues that should be addressed first, call it out. If ERRORS.md doesn't exist, note it as a finding ("no code audit was run").
|
|
46
|
+
|
|
47
|
+
1c. **Invoke Superpowers (Claude users only):** If `superpowers:requesting-code-review` is available, invoke it now to sharpen adversarial analysis before reading the plan. If not available, proceed with native capabilities.
|
|
46
48
|
|
|
47
49
|
2. **Read the actual codebase** — don't review a plan in a vacuum. Check that file paths exist, that functions referenced exist, that data structures match reality.
|
|
48
50
|
|
package/skills/the-inspector.md
CHANGED
|
@@ -60,10 +60,12 @@ There is no partial pass. The build either passes inspection or it goes back to
|
|
|
60
60
|
## Process
|
|
61
61
|
|
|
62
62
|
1. Read `PLAN.md` — know the contract
|
|
63
|
-
2. Read
|
|
64
|
-
3.
|
|
65
|
-
4.
|
|
66
|
-
5.
|
|
63
|
+
2. Read `ERRORS.md` — know the pre-existing issues
|
|
64
|
+
3. Read every file that was modified
|
|
65
|
+
4. Work through the test matrix item by item
|
|
66
|
+
5. **Update `ERRORS.md`** — mark any issues that were fixed this session as resolved. Add any new issues discovered during verification. Never delete old entries — mark them with `[RESOLVED — date]`.
|
|
67
|
+
6. Report: PASS (all items verified, user confirmed) or FAIL (specific items failing + fix list)
|
|
68
|
+
7. On PASS → tell The Leader: inspection complete, ready for handoff
|
|
67
69
|
|
|
68
70
|
## Rules
|
|
69
71
|
|
package/skills/the-leader.md
CHANGED
|
@@ -24,8 +24,16 @@ You are The Leader. You own the pipeline from task definition to handoff. Every
|
|
|
24
24
|
- Verify PLAN.md exists and has no TBD/TODO/❓ markers before passing to Challenger
|
|
25
25
|
- If markers exist → send back to Visionary. Do not pass incomplete plans.
|
|
26
26
|
|
|
27
|
+
**Phase 1b — Code Audit (auto):**
|
|
28
|
+
- Check if `ERRORS.md` exists in `~/.g2w/projects/[active-project]/`
|
|
29
|
+
- If it doesn't exist OR is older than 7 days → run a full code audit now
|
|
30
|
+
- Scan the codebase for: bugs, crashes, race conditions, dead code, anti-patterns, security issues
|
|
31
|
+
- Write findings to `ERRORS.md` with file paths, line numbers, and specific descriptions
|
|
32
|
+
- This runs automatically — never ask the user if they want an audit. They always do.
|
|
33
|
+
- If `ERRORS.md` already exists and is current → skip, move on
|
|
34
|
+
|
|
27
35
|
**Phase 2 — Challenge:**
|
|
28
|
-
- Hand plan to The Challenger
|
|
36
|
+
- Hand plan AND `ERRORS.md` to The Challenger
|
|
29
37
|
- Receive findings
|
|
30
38
|
- If FAIL → return to Visionary with findings. Loop until Challenger gives PASS.
|
|
31
39
|
- On PASS → LOCK the plan. Announce: "Plan is locked. No modifications after this point."
|