@leejungkiin/awkit 1.3.8 → 1.4.2
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/bin/awk.js +630 -52
- package/bin/claude-generators.js +122 -0
- package/core/AGENTS.md +54 -0
- package/core/CLAUDE.md +155 -0
- package/core/GEMINI.md +44 -9
- package/core/GEMINI.md.bak +126 -199
- package/package.json +1 -1
- package/skills/ai-sprite-maker/SKILL.md +81 -0
- package/skills/ai-sprite-maker/scripts/animate_sprite.py +102 -0
- package/skills/ai-sprite-maker/scripts/process_sprites.py +140 -0
- package/skills/awf-session-restore/SKILL.md +12 -2
- package/skills/brainstorm-agent/SKILL.md +11 -8
- package/skills/code-review/SKILL.md +21 -33
- package/skills/gitnexus/gitnexus-cli/SKILL.md +82 -0
- package/skills/gitnexus/gitnexus-debugging/SKILL.md +89 -0
- package/skills/gitnexus/gitnexus-exploring/SKILL.md +78 -0
- package/skills/gitnexus/gitnexus-guide/SKILL.md +64 -0
- package/skills/gitnexus/gitnexus-impact-analysis/SKILL.md +97 -0
- package/skills/gitnexus/gitnexus-refactoring/SKILL.md +121 -0
- package/skills/lucylab-tts/SKILL.md +64 -0
- package/skills/lucylab-tts/resources/voices_library.json +908 -0
- package/skills/lucylab-tts/scripts/.env +1 -0
- package/skills/lucylab-tts/scripts/lucylab_tts.py +506 -0
- package/skills/nm-memory-sync/SKILL.md +14 -1
- package/skills/orchestrator/SKILL.md +5 -38
- package/skills/ship-to-code/SKILL.md +115 -0
- package/skills/short-maker/SKILL.md +150 -0
- package/skills/short-maker/_backup/storyboard.html +106 -0
- package/skills/short-maker/_backup/video_mixer.py +296 -0
- package/skills/short-maker/outputs/fitbite-promo/background.jpg +0 -0
- package/skills/short-maker/outputs/fitbite-promo/final/promo-final.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/script.md +19 -0
- package/skills/short-maker/outputs/fitbite-promo/segments/scene-01.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/segments/scene-02.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/segments/scene-03.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/segments/scene-04.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/storyboard/scene-01.png +0 -0
- package/skills/short-maker/outputs/fitbite-promo/storyboard/scene-02.png +0 -0
- package/skills/short-maker/outputs/fitbite-promo/storyboard/scene-03.png +0 -0
- package/skills/short-maker/outputs/fitbite-promo/storyboard/scene-04.png +0 -0
- package/skills/short-maker/outputs/fitbite-promo/storyboard.html +133 -0
- package/skills/short-maker/outputs/fitbite-promo/storyboard.json +38 -0
- package/skills/short-maker/outputs/fitbite-promo/temp/merged_chroma.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/temp/merged_crossfaded.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/temp/ready_00.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/temp/ready_01.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/temp/ready_02.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/temp/ready_03.mp4 +0 -0
- package/skills/short-maker/outputs/fitbite-promo/tts/manifest.json +31 -0
- package/skills/short-maker/outputs/fitbite-promo/tts/scene-01.wav +0 -0
- package/skills/short-maker/outputs/fitbite-promo/tts/scene-02.wav +0 -0
- package/skills/short-maker/outputs/fitbite-promo/tts/scene-03.wav +0 -0
- package/skills/short-maker/outputs/fitbite-promo/tts/scene-04.wav +0 -0
- package/skills/short-maker/outputs/fitbite-promo/tts_script.txt +11 -0
- package/skills/short-maker/scripts/google-flow-cli/.project-identity +41 -0
- package/skills/short-maker/scripts/google-flow-cli/.trae/rules/project_rules.md +52 -0
- package/skills/short-maker/scripts/google-flow-cli/CODEBASE.md +67 -0
- package/skills/short-maker/scripts/google-flow-cli/GoogleFlowCli.code-workspace +29 -0
- package/skills/short-maker/scripts/google-flow-cli/README.md +168 -0
- package/skills/short-maker/scripts/google-flow-cli/docs/specs/PROJECT.md +12 -0
- package/skills/short-maker/scripts/google-flow-cli/docs/specs/REQUIREMENTS.md +22 -0
- package/skills/short-maker/scripts/google-flow-cli/docs/specs/ROADMAP.md +16 -0
- package/skills/short-maker/scripts/google-flow-cli/docs/specs/TECH-SPEC.md +13 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/__init__.py +3 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/api/__init__.py +19 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/api/client.py +1921 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/api/models.py +64 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/api/rpc_ids.py +98 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/auth/__init__.py +15 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/auth/browser_auth.py +692 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/auth/humanizer.py +417 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/auth/proxy_ext.py +120 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/auth/recaptcha.py +482 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/batchexecute/__init__.py +5 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/batchexecute/client.py +414 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/cli/__init__.py +1 -0
- package/skills/short-maker/scripts/google-flow-cli/gflow/cli/main.py +1075 -0
- package/skills/short-maker/scripts/google-flow-cli/pyproject.toml +36 -0
- package/skills/short-maker/scripts/google-flow-cli/script.txt +22 -0
- package/skills/short-maker/scripts/google-flow-cli/tests/__init__.py +0 -0
- package/skills/short-maker/scripts/google-flow-cli/tests/test_batchexecute.py +113 -0
- package/skills/short-maker/scripts/google-flow-cli/tests/test_client.py +190 -0
- package/skills/short-maker/templates/aida_script.md +40 -0
- package/skills/short-maker/templates/mimic_analyzer.md +29 -0
- package/skills/single-flow-task-execution/SKILL.md +412 -0
- package/skills/single-flow-task-execution/code-quality-reviewer-prompt.md +20 -0
- package/skills/single-flow-task-execution/implementer-prompt.md +78 -0
- package/skills/single-flow-task-execution/spec-reviewer-prompt.md +61 -0
- package/skills/skill-creator/SKILL.md +44 -0
- package/skills/spm-build-analysis/SKILL.md +92 -0
- package/skills/spm-build-analysis/references/build-optimization-sources.md +155 -0
- package/skills/spm-build-analysis/references/recommendation-format.md +85 -0
- package/skills/spm-build-analysis/references/spm-analysis-checks.md +105 -0
- package/skills/spm-build-analysis/scripts/check_spm_pins.py +118 -0
- package/skills/symphony-enforcer/SKILL.md +83 -97
- package/skills/symphony-orchestrator/SKILL.md +1 -1
- package/skills/trello-sync/SKILL.md +52 -45
- package/skills/verification-gate/SKILL.md +13 -2
- package/skills/xcode-build-benchmark/SKILL.md +88 -0
- package/skills/xcode-build-benchmark/references/benchmark-artifacts.md +94 -0
- package/skills/xcode-build-benchmark/references/benchmarking-workflow.md +67 -0
- package/skills/xcode-build-benchmark/schemas/build-benchmark.schema.json +230 -0
- package/skills/xcode-build-benchmark/scripts/benchmark_builds.py +308 -0
- package/skills/xcode-build-fixer/SKILL.md +218 -0
- package/skills/xcode-build-fixer/references/build-settings-best-practices.md +216 -0
- package/skills/xcode-build-fixer/references/fix-patterns.md +290 -0
- package/skills/xcode-build-fixer/references/recommendation-format.md +85 -0
- package/skills/xcode-build-fixer/scripts/benchmark_builds.py +308 -0
- package/skills/xcode-build-orchestrator/SKILL.md +156 -0
- package/skills/xcode-build-orchestrator/references/benchmark-artifacts.md +94 -0
- package/skills/xcode-build-orchestrator/references/build-settings-best-practices.md +216 -0
- package/skills/xcode-build-orchestrator/references/orchestration-report-template.md +143 -0
- package/skills/xcode-build-orchestrator/references/recommendation-format.md +85 -0
- package/skills/xcode-build-orchestrator/scripts/benchmark_builds.py +308 -0
- package/skills/xcode-build-orchestrator/scripts/diagnose_compilation.py +273 -0
- package/skills/xcode-build-orchestrator/scripts/generate_optimization_report.py +533 -0
- package/skills/xcode-compilation-analyzer/SKILL.md +89 -0
- package/skills/xcode-compilation-analyzer/references/build-optimization-sources.md +155 -0
- package/skills/xcode-compilation-analyzer/references/code-compilation-checks.md +106 -0
- package/skills/xcode-compilation-analyzer/references/recommendation-format.md +85 -0
- package/skills/xcode-compilation-analyzer/scripts/diagnose_compilation.py +273 -0
- package/skills/xcode-project-analyzer/SKILL.md +76 -0
- package/skills/xcode-project-analyzer/references/build-optimization-sources.md +155 -0
- package/skills/xcode-project-analyzer/references/build-settings-best-practices.md +216 -0
- package/skills/xcode-project-analyzer/references/project-audit-checks.md +101 -0
- package/skills/xcode-project-analyzer/references/recommendation-format.md +85 -0
- package/templates/CODEBASE.md +26 -42
- package/templates/configs/trello-config.json +2 -2
- package/templates/workflow_dual_mode_template.md +5 -5
- package/workflows/_uncategorized/conductor-codex.md +125 -0
- package/workflows/_uncategorized/conductor.md +97 -0
- package/workflows/_uncategorized/ship-to-code.md +85 -0
- package/workflows/_uncategorized/trello-sync.md +52 -0
- package/workflows/context/codebase-sync.md +10 -87
- package/workflows/quality/visual-debug.md +66 -12
|
@@ -0,0 +1,412 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: single-flow-task-execution
|
|
3
|
+
description: Use when executing implementation plans, handling multiple independent tasks, or doing structured task-by-task development with review gates in Antigravity.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Single-Flow Task Execution
|
|
7
|
+
|
|
8
|
+
Execute plans by working through one task at a time with two-stage review after each: spec compliance review first, then code quality review.
|
|
9
|
+
|
|
10
|
+
**Core principle:** One task at a time + two-stage review (spec then quality) = high quality, disciplined iteration.
|
|
11
|
+
|
|
12
|
+
## Antigravity Execution Model
|
|
13
|
+
|
|
14
|
+
Antigravity does NOT support parallel coding subagents. All work happens in a single execution thread.
|
|
15
|
+
|
|
16
|
+
**Rules:**
|
|
17
|
+
|
|
18
|
+
1. **One active task only** — never work on multiple tasks simultaneously.
|
|
19
|
+
2. **One execution thread only** — no parallel dispatch.
|
|
20
|
+
3. **No parallel coding subagents** — Antigravity does not have `Task(...)`.
|
|
21
|
+
4. **Browser automation** may use `browser_subagent` in isolated steps.
|
|
22
|
+
5. **Track progress** via Symphony task system. Update task status at each state change.
|
|
23
|
+
6. **Use `task_boundary`** to clearly delineate each unit of work.
|
|
24
|
+
|
|
25
|
+
## When to Use
|
|
26
|
+
|
|
27
|
+
```dot
|
|
28
|
+
digraph when_to_use {
|
|
29
|
+
"Have implementation plan?" [shape=diamond];
|
|
30
|
+
"Tasks mostly independent?" [shape=diamond];
|
|
31
|
+
"Multiple problems to solve?" [shape=diamond];
|
|
32
|
+
"single-flow-task-execution" [shape=box];
|
|
33
|
+
"executing-plans" [shape=box];
|
|
34
|
+
"Manual execution or brainstorm first" [shape=box];
|
|
35
|
+
|
|
36
|
+
"Have implementation plan?" -> "Tasks mostly independent?" [label="yes"];
|
|
37
|
+
"Have implementation plan?" -> "Manual execution or brainstorm first" [label="no"];
|
|
38
|
+
"Tasks mostly independent?" -> "single-flow-task-execution" [label="yes"];
|
|
39
|
+
"Tasks mostly independent?" -> "Manual execution or brainstorm first" [label="no - tightly coupled"];
|
|
40
|
+
"Multiple problems to solve?" -> "single-flow-task-execution" [label="yes - work through them sequentially"];
|
|
41
|
+
"Multiple problems to solve?" -> "Manual execution or brainstorm first" [label="no - single task"];
|
|
42
|
+
}
|
|
43
|
+
```
|
|
44
|
+
|
|
45
|
+
**Use when:**
|
|
46
|
+
|
|
47
|
+
- You have an implementation plan with multiple independent tasks
|
|
48
|
+
- 2+ test files failing with different root causes (work through them one at a time)
|
|
49
|
+
- Multiple subsystems broken independently
|
|
50
|
+
- Each problem can be understood without context from others
|
|
51
|
+
- Structured execution with quality gates is needed
|
|
52
|
+
|
|
53
|
+
**Don't use when:**
|
|
54
|
+
|
|
55
|
+
- Failures are related (fix one might fix others) — investigate together first
|
|
56
|
+
- Tasks are tightly coupled and need full system understanding
|
|
57
|
+
- Single simple task that doesn't need review structure
|
|
58
|
+
|
|
59
|
+
**vs. Executing Plans (worktree-based):**
|
|
60
|
+
|
|
61
|
+
- Same session (no context switch)
|
|
62
|
+
- Fresh `task_boundary` per task (clean scope)
|
|
63
|
+
- Two-stage review after each task: spec compliance first, then code quality
|
|
64
|
+
- Faster iteration (no human-in-loop between tasks)
|
|
65
|
+
|
|
66
|
+
## The Process
|
|
67
|
+
|
|
68
|
+
```dot
|
|
69
|
+
digraph process {
|
|
70
|
+
rankdir=TB;
|
|
71
|
+
|
|
72
|
+
subgraph cluster_per_task {
|
|
73
|
+
label="Per Task";
|
|
74
|
+
"Execute implementation (./implementer-prompt.md)" [shape=box];
|
|
75
|
+
"Questions about requirements?" [shape=diamond];
|
|
76
|
+
"Answer questions, provide context" [shape=box];
|
|
77
|
+
"Implement, test, commit, self-review" [shape=box];
|
|
78
|
+
"Run spec compliance review (./spec-reviewer-prompt.md)" [shape=box];
|
|
79
|
+
"Spec confirms code matches spec?" [shape=diamond];
|
|
80
|
+
"Fix spec gaps" [shape=box];
|
|
81
|
+
"Run code quality review (./code-quality-reviewer-prompt.md)" [shape=box];
|
|
82
|
+
"Code quality approved?" [shape=diamond];
|
|
83
|
+
"Fix quality issues" [shape=box];
|
|
84
|
+
"Mark task complete via Symphony" [shape=box];
|
|
85
|
+
}
|
|
86
|
+
|
|
87
|
+
"Read plan, extract all tasks with full text, note context" [shape=box];
|
|
88
|
+
"More tasks remain?" [shape=diamond];
|
|
89
|
+
"Run final code review for entire implementation" [shape=box];
|
|
90
|
+
"Complete Symphony task and present next steps" [shape=box style=filled fillcolor=lightgreen];
|
|
91
|
+
|
|
92
|
+
"Read plan, extract all tasks with full text, note context" -> "Execute implementation (./implementer-prompt.md)";
|
|
93
|
+
"Execute implementation (./implementer-prompt.md)" -> "Questions about requirements?";
|
|
94
|
+
"Questions about requirements?" -> "Answer questions, provide context" [label="yes"];
|
|
95
|
+
"Answer questions, provide context" -> "Execute implementation (./implementer-prompt.md)";
|
|
96
|
+
"Questions about requirements?" -> "Implement, test, commit, self-review" [label="no"];
|
|
97
|
+
"Implement, test, commit, self-review" -> "Run spec compliance review (./spec-reviewer-prompt.md)";
|
|
98
|
+
"Run spec compliance review (./spec-reviewer-prompt.md)" -> "Spec confirms code matches spec?";
|
|
99
|
+
"Spec confirms code matches spec?" -> "Fix spec gaps" [label="no"];
|
|
100
|
+
"Fix spec gaps" -> "Run spec compliance review (./spec-reviewer-prompt.md)" [label="re-review"];
|
|
101
|
+
"Spec confirms code matches spec?" -> "Run code quality review (./code-quality-reviewer-prompt.md)" [label="yes"];
|
|
102
|
+
"Run code quality review (./code-quality-reviewer-prompt.md)" -> "Code quality approved?";
|
|
103
|
+
"Code quality approved?" -> "Fix quality issues" [label="no"];
|
|
104
|
+
"Fix quality issues" -> "Run code quality review (./code-quality-reviewer-prompt.md)" [label="re-review"];
|
|
105
|
+
"Code quality approved?" -> "Mark task complete via Symphony" [label="yes"];
|
|
106
|
+
"Mark task complete via Symphony" -> "More tasks remain?";
|
|
107
|
+
"More tasks remain?" -> "Execute implementation (./implementer-prompt.md)" [label="yes"];
|
|
108
|
+
"More tasks remain?" -> "Run final code review for entire implementation" [label="no"];
|
|
109
|
+
"Run final code review for entire implementation" -> "Complete Symphony task and present next steps";
|
|
110
|
+
}
|
|
111
|
+
```
|
|
112
|
+
|
|
113
|
+
## UI-First Task Ordering (Gate 4 Three-Phase — v12.3)
|
|
114
|
+
|
|
115
|
+
When a task set includes UI components (COMPLEX or MODERATE), tasks MUST be ordered in three phases:
|
|
116
|
+
|
|
117
|
+
### Phase A: Infrastructure Tasks
|
|
118
|
+
```
|
|
119
|
+
Priority: Execute FIRST
|
|
120
|
+
Examples:
|
|
121
|
+
- Add dependencies (Gradle, SPM, CocoaPods)
|
|
122
|
+
- Create project structure (packages, modules, DI)
|
|
123
|
+
- Set up navigation skeleton (NavGraph, Router)
|
|
124
|
+
- Configure build variants, signing
|
|
125
|
+
Gate: App MUST build successfully → proceed
|
|
126
|
+
```
|
|
127
|
+
|
|
128
|
+
### Phase B: UI Shell Tasks (Mock Data)
|
|
129
|
+
```
|
|
130
|
+
Priority: Execute SECOND, BEFORE any logic tasks
|
|
131
|
+
Examples:
|
|
132
|
+
- Build all screen layouts with static/mock data
|
|
133
|
+
- Implement navigation between screens
|
|
134
|
+
- Add animations, transitions, loading/empty/error states
|
|
135
|
+
- Wire up UI components (no real API/DB calls)
|
|
136
|
+
Gate: 🧪 USER TEST CHECKPOINT or 🤖 AUTO DEVICE CHECKPOINT
|
|
137
|
+
→ Check `.project-identity` for `autoVerification` flag.
|
|
138
|
+
→ If `false` (default): Stop & block user for manual test (BlockedOnUser=true).
|
|
139
|
+
→ If `true`: AI auto-tests on device (Maestro MCP) & takes UI screenshot. Proceed to Phase C WITHOUT blocking user if build/launch OK.
|
|
140
|
+
```
|
|
141
|
+
|
|
142
|
+
### Phase C: Logic Tasks (Per Feature)
|
|
143
|
+
```
|
|
144
|
+
Priority: Execute LAST, after UI is confirmed
|
|
145
|
+
Examples:
|
|
146
|
+
- Replace mock data with real API/DB calls
|
|
147
|
+
- Implement business logic, validation
|
|
148
|
+
- Add error handling, retry, offline support
|
|
149
|
+
- Wire up hardware features (camera, GPS, sensors)
|
|
150
|
+
Gate: 🧪 MANUAL or 🤖 AUTO CHECKPOINT per feature (batch small ones)
|
|
151
|
+
→ Depends on `autoVerification` setting. Auto uses Build & Maestro. Manual waits for user confirmation.
|
|
152
|
+
```
|
|
153
|
+
|
|
154
|
+
### Task Sorting Rule
|
|
155
|
+
```
|
|
156
|
+
When creating task list from implementation plan:
|
|
157
|
+
1. Tag each task: [INFRA] [UI] [LOGIC]
|
|
158
|
+
2. Sort: INFRA first → UI second → LOGIC last
|
|
159
|
+
3. Within each phase: respect dependency ordering
|
|
160
|
+
4. Between phases: MANDATORY checkpoint where indicated
|
|
161
|
+
```
|
|
162
|
+
|
|
163
|
+
## Task Decomposition
|
|
164
|
+
|
|
165
|
+
When facing multiple problems (e.g., 5 test failures across 3 files):
|
|
166
|
+
|
|
167
|
+
### 1. Identify Independent Domains
|
|
168
|
+
|
|
169
|
+
Group failures by what's broken:
|
|
170
|
+
|
|
171
|
+
- File A tests: User authentication flow
|
|
172
|
+
- File B tests: Data validation logic
|
|
173
|
+
- File C tests: API response handling
|
|
174
|
+
|
|
175
|
+
Each domain is independent — fixing authentication doesn't affect validation tests.
|
|
176
|
+
|
|
177
|
+
### 2. Create Task Units
|
|
178
|
+
|
|
179
|
+
Each task gets:
|
|
180
|
+
|
|
181
|
+
- **Specific scope:** One test file or subsystem
|
|
182
|
+
- **Clear goal:** Make these tests pass / implement this feature
|
|
183
|
+
- **Constraints:** Don't change unrelated code
|
|
184
|
+
- **Expected output:** Summary of what changed and verification results
|
|
185
|
+
|
|
186
|
+
### 3. Execute Sequentially with Review
|
|
187
|
+
|
|
188
|
+
Work through each task one at a time using the full review cycle.
|
|
189
|
+
|
|
190
|
+
### 4. Review and Integrate
|
|
191
|
+
|
|
192
|
+
After all tasks:
|
|
193
|
+
|
|
194
|
+
- Run full test suite to verify no regressions
|
|
195
|
+
- Check for conflicts between task changes
|
|
196
|
+
- Run final code review on entire implementation
|
|
197
|
+
|
|
198
|
+
## Task Brief Structure
|
|
199
|
+
|
|
200
|
+
For each task, prepare:
|
|
201
|
+
|
|
202
|
+
```
|
|
203
|
+
task_boundary:
|
|
204
|
+
description: "Implement Task N: [task name]"
|
|
205
|
+
prompt: |
|
|
206
|
+
## Task Description
|
|
207
|
+
[FULL TEXT of task from plan — paste it here]
|
|
208
|
+
|
|
209
|
+
## Context
|
|
210
|
+
[Where this fits, dependencies, architectural context]
|
|
211
|
+
|
|
212
|
+
## Constraints
|
|
213
|
+
- Only modify [specific files/directories]
|
|
214
|
+
- Follow existing patterns in the codebase
|
|
215
|
+
- Write tests for new functionality
|
|
216
|
+
|
|
217
|
+
## Verification
|
|
218
|
+
- Run: [specific test command]
|
|
219
|
+
- Expected: [what success looks like]
|
|
220
|
+
```
|
|
221
|
+
|
|
222
|
+
**Key:** Provide full task text and context upfront. Don't make the task boundary re-read the plan file.
|
|
223
|
+
|
|
224
|
+
## Review Templates
|
|
225
|
+
|
|
226
|
+
This skill includes prompt templates for structured reviews:
|
|
227
|
+
|
|
228
|
+
- **`./implementer-prompt.md`** — Template for implementation task boundaries
|
|
229
|
+
- **`./spec-reviewer-prompt.md`** — Template for spec compliance review (did we build what was requested?)
|
|
230
|
+
- **`./code-quality-reviewer-prompt.md`** — Template for code quality review (is it well-built?)
|
|
231
|
+
|
|
232
|
+
**Review order matters:** Always run spec compliance FIRST, then code quality. There's no point reviewing code quality if the implementation doesn't match the spec.
|
|
233
|
+
|
|
234
|
+
## Checkpoint Pattern
|
|
235
|
+
|
|
236
|
+
At logical boundaries (after each task, at major milestones), report:
|
|
237
|
+
|
|
238
|
+
- **What changed** — files modified, features implemented
|
|
239
|
+
- **What verification ran** — test results, lint results
|
|
240
|
+
- **What remains** — remaining tasks, known issues
|
|
241
|
+
|
|
242
|
+
Update Symphony task progress with current status.
|
|
243
|
+
**Crucially (Inter-task Flow):**
|
|
244
|
+
- If `.project-identity` has `"autoVerification": true`: Do NOT block the user (`BlockedOnUser=false`) between tasks/batches. Auto-proceed immediately to the next task if auto-verification (Maestro) passes.
|
|
245
|
+
- If `"autoVerification": false` (default): MUST stop and wait for user's manual validation (`BlockedOnUser=true`) before proceeding to the next batch.
|
|
246
|
+
|
|
247
|
+
## Common Mistakes
|
|
248
|
+
|
|
249
|
+
**Task scoping:**
|
|
250
|
+
|
|
251
|
+
- **Bad:** "Fix all the tests" — loses focus
|
|
252
|
+
- **Good:** "Fix user-auth.test.ts failures" — clear scope
|
|
253
|
+
|
|
254
|
+
**Context:**
|
|
255
|
+
|
|
256
|
+
- **Bad:** "Fix the validation bug" — unclear where
|
|
257
|
+
- **Good:** Paste error messages, test names, relevant code paths
|
|
258
|
+
|
|
259
|
+
**Constraints:**
|
|
260
|
+
|
|
261
|
+
- **Bad:** No constraints — task might refactor everything
|
|
262
|
+
- **Good:** "Only modify src/auth/ directory"
|
|
263
|
+
|
|
264
|
+
**Output:**
|
|
265
|
+
|
|
266
|
+
- **Bad:** "Fix it" — no visibility into what changed
|
|
267
|
+
- **Good:** "Report: root cause, changes made, test results"
|
|
268
|
+
|
|
269
|
+
**Reviews:**
|
|
270
|
+
|
|
271
|
+
- **Bad:** "It works, move on" — quality debt
|
|
272
|
+
- **Good:** Implement then spec review then quality review then next task
|
|
273
|
+
|
|
274
|
+
## Example Workflow
|
|
275
|
+
|
|
276
|
+
```
|
|
277
|
+
You: I'm using single-flow-task-execution to execute this plan.
|
|
278
|
+
|
|
279
|
+
[Read plan file: docs/plans/feature-plan.md]
|
|
280
|
+
[Extract all 5 tasks with full text and context]
|
|
281
|
+
[Create Symphony tasks for tracking]
|
|
282
|
+
|
|
283
|
+
--- Task 1: Hook installation script ---
|
|
284
|
+
|
|
285
|
+
[Prepare task brief with full text + context]
|
|
286
|
+
[Execute implementation following ./implementer-prompt.md structure]
|
|
287
|
+
|
|
288
|
+
Questions: "Should the hook be installed at user or system level?"
|
|
289
|
+
Answer: "User level (~/.config/superpowers/hooks/)"
|
|
290
|
+
|
|
291
|
+
Implementation:
|
|
292
|
+
- Implemented install-hook command
|
|
293
|
+
- Added tests, 5/5 passing
|
|
294
|
+
- Self-review: Found I missed --force flag, added it
|
|
295
|
+
- Committed
|
|
296
|
+
|
|
297
|
+
[Run spec compliance review following ./spec-reviewer-prompt.md]
|
|
298
|
+
Spec review: Spec compliant — all requirements met, nothing extra
|
|
299
|
+
|
|
300
|
+
[Run code quality review following ./code-quality-reviewer-prompt.md]
|
|
301
|
+
Code review: Strengths: Good test coverage, clean. Issues: None. Approved.
|
|
302
|
+
|
|
303
|
+
[Mark Task 1 complete in Symphony]
|
|
304
|
+
|
|
305
|
+
--- Task 2: Recovery modes ---
|
|
306
|
+
|
|
307
|
+
[Prepare task brief with full text + context]
|
|
308
|
+
[Execute implementation]
|
|
309
|
+
|
|
310
|
+
Implementation:
|
|
311
|
+
- Added verify/repair modes
|
|
312
|
+
- 8/8 tests passing
|
|
313
|
+
- Self-review: All good
|
|
314
|
+
- Committed
|
|
315
|
+
|
|
316
|
+
[Run spec compliance review]
|
|
317
|
+
Spec review: Issues found:
|
|
318
|
+
- Missing: Progress reporting (spec says "report every 100 items")
|
|
319
|
+
- Extra: Added --json flag (not requested)
|
|
320
|
+
|
|
321
|
+
[Fix issues: remove --json flag, add progress reporting]
|
|
322
|
+
[Run spec compliance review again]
|
|
323
|
+
Spec review: Spec compliant now
|
|
324
|
+
|
|
325
|
+
[Run code quality review]
|
|
326
|
+
Code review: Issue (Important): Magic number (100) should be a constant
|
|
327
|
+
|
|
328
|
+
[Fix: extract PROGRESS_INTERVAL constant]
|
|
329
|
+
[Run code quality review again]
|
|
330
|
+
Code review: Approved
|
|
331
|
+
|
|
332
|
+
[Mark Task 2 complete in Symphony]
|
|
333
|
+
|
|
334
|
+
... [Continue through remaining tasks] ...
|
|
335
|
+
|
|
336
|
+
[After all tasks complete]
|
|
337
|
+
[Run final code review on entire implementation]
|
|
338
|
+
Final review: All requirements met, ready to merge
|
|
339
|
+
|
|
340
|
+
[Complete Symphony task and present next steps]
|
|
341
|
+
Done!
|
|
342
|
+
```
|
|
343
|
+
|
|
344
|
+
## Red Flags
|
|
345
|
+
|
|
346
|
+
**Never:**
|
|
347
|
+
|
|
348
|
+
- Start implementation on main/master branch without explicit user consent
|
|
349
|
+
- Skip reviews (spec compliance OR code quality)
|
|
350
|
+
- Proceed with unfixed review issues
|
|
351
|
+
- Work on multiple tasks simultaneously
|
|
352
|
+
- Skip scene-setting context (task needs to understand where it fits)
|
|
353
|
+
- Accept "close enough" on spec compliance (reviewer found issues = not done)
|
|
354
|
+
- Skip review loops (reviewer found issues = fix = review again)
|
|
355
|
+
- Let self-review replace actual review (both are needed)
|
|
356
|
+
- **Start code quality review before spec compliance passes** (wrong order)
|
|
357
|
+
- Move to next task while either review has open issues
|
|
358
|
+
|
|
359
|
+
**If you have questions about requirements:**
|
|
360
|
+
|
|
361
|
+
- Ask clearly and wait for answers
|
|
362
|
+
- Don't guess or make assumptions
|
|
363
|
+
- Better to ask upfront than rework later
|
|
364
|
+
|
|
365
|
+
**If reviewer finds issues:**
|
|
366
|
+
|
|
367
|
+
- Fix them
|
|
368
|
+
- Run reviewer again
|
|
369
|
+
- Repeat until approved
|
|
370
|
+
- Don't skip the re-review
|
|
371
|
+
|
|
372
|
+
## Completion
|
|
373
|
+
|
|
374
|
+
Before claiming all work is done:
|
|
375
|
+
|
|
376
|
+
1. Ensure all Symphony tasks are marked `done` or `cancelled`
|
|
377
|
+
2. Run full test/validation command
|
|
378
|
+
3. Verify no regressions across all tasks
|
|
379
|
+
4. Summarize evidence (test output, review approvals)
|
|
380
|
+
|
|
381
|
+
## Advantages
|
|
382
|
+
|
|
383
|
+
**Structured execution:**
|
|
384
|
+
|
|
385
|
+
- Clear task boundaries prevent scope creep
|
|
386
|
+
- Review gates catch issues early (cheaper than debugging later)
|
|
387
|
+
- Progress tracking provides visibility
|
|
388
|
+
|
|
389
|
+
**Quality gates:**
|
|
390
|
+
|
|
391
|
+
- Self-review catches obvious issues before handoff
|
|
392
|
+
- Two-stage review: spec compliance prevents over/under-building, code quality ensures maintainability
|
|
393
|
+
- Review loops ensure fixes actually work
|
|
394
|
+
|
|
395
|
+
**Efficiency:**
|
|
396
|
+
|
|
397
|
+
- Provide full task text upfront (no re-reading plan files)
|
|
398
|
+
- Controller curates exactly what context is needed
|
|
399
|
+
- Questions surfaced before work begins (not after)
|
|
400
|
+
- Sequential execution avoids conflicts between tasks
|
|
401
|
+
|
|
402
|
+
## Integration
|
|
403
|
+
|
|
404
|
+
**Required workflow skills:**
|
|
405
|
+
|
|
406
|
+
- **`~/.gemini/antigravity/skills/symphony-orchestrator/SKILL.md`** — Task tracking and lifecycle
|
|
407
|
+
- **`~/.gemini/antigravity/skills/symphony-enforcer/SKILL.md`** — Enforce task discipline
|
|
408
|
+
|
|
409
|
+
**Should also use:**
|
|
410
|
+
|
|
411
|
+
- **test-driven-development** — Follow TDD for each task
|
|
412
|
+
- **verification-before-completion** — Final verification checklist
|
|
@@ -0,0 +1,20 @@
|
|
|
1
|
+
# Code Quality Reviewer Prompt Template
|
|
2
|
+
|
|
3
|
+
Use this template when running a code quality review step in single-flow mode.
|
|
4
|
+
|
|
5
|
+
**Purpose:** Verify implementation is well-built (clean, tested, maintainable)
|
|
6
|
+
|
|
7
|
+
**Only proceed after spec compliance review passes.**
|
|
8
|
+
|
|
9
|
+
```
|
|
10
|
+
task_boundary:
|
|
11
|
+
Use template at requesting-code-review/code-reviewer.md
|
|
12
|
+
|
|
13
|
+
WHAT_WAS_IMPLEMENTED: [from implementer's report]
|
|
14
|
+
PLAN_OR_REQUIREMENTS: Task N from [plan-file]
|
|
15
|
+
BASE_SHA: [commit before task]
|
|
16
|
+
HEAD_SHA: [current commit]
|
|
17
|
+
DESCRIPTION: [task summary]
|
|
18
|
+
```
|
|
19
|
+
|
|
20
|
+
**Code reviewer returns:** Strengths, Issues (Critical/Important/Minor), Assessment
|
|
@@ -0,0 +1,78 @@
|
|
|
1
|
+
# Implementer Task Template
|
|
2
|
+
|
|
3
|
+
Use this template when executing an implementation task in single-flow mode.
|
|
4
|
+
|
|
5
|
+
```
|
|
6
|
+
task_boundary:
|
|
7
|
+
description: "Implement Task N: [task name]"
|
|
8
|
+
prompt: |
|
|
9
|
+
You are implementing Task N: [task name]
|
|
10
|
+
|
|
11
|
+
## Task Description
|
|
12
|
+
|
|
13
|
+
[FULL TEXT of task from plan - paste it here]
|
|
14
|
+
|
|
15
|
+
## Context
|
|
16
|
+
|
|
17
|
+
[Scene-setting: where this fits, dependencies, architectural context]
|
|
18
|
+
|
|
19
|
+
## Before You Begin
|
|
20
|
+
|
|
21
|
+
If you have questions about:
|
|
22
|
+
- The requirements or acceptance criteria
|
|
23
|
+
- The approach or implementation strategy
|
|
24
|
+
- Dependencies or assumptions
|
|
25
|
+
- Anything unclear in the task description
|
|
26
|
+
|
|
27
|
+
**Ask them now.** Raise any concerns before starting work.
|
|
28
|
+
|
|
29
|
+
## Your Job
|
|
30
|
+
|
|
31
|
+
Once you're clear on requirements:
|
|
32
|
+
1. Implement exactly what the task specifies
|
|
33
|
+
2. Write tests (following TDD if task says to)
|
|
34
|
+
3. Verify implementation works
|
|
35
|
+
4. Commit your work
|
|
36
|
+
5. Self-review (see below)
|
|
37
|
+
6. Report back
|
|
38
|
+
|
|
39
|
+
Work from: [directory]
|
|
40
|
+
|
|
41
|
+
**While you work:** If you encounter something unexpected or unclear, **ask questions**.
|
|
42
|
+
It's always OK to pause and clarify. Don't guess or make assumptions.
|
|
43
|
+
|
|
44
|
+
## Before Reporting Back: Self-Review
|
|
45
|
+
|
|
46
|
+
Review your work with fresh eyes. Ask yourself:
|
|
47
|
+
|
|
48
|
+
**Completeness:**
|
|
49
|
+
- Did I fully implement everything in the spec?
|
|
50
|
+
- Did I miss any requirements?
|
|
51
|
+
- Are there edge cases I didn't handle?
|
|
52
|
+
|
|
53
|
+
**Quality:**
|
|
54
|
+
- Is this my best work?
|
|
55
|
+
- Are names clear and accurate (match what things do, not how they work)?
|
|
56
|
+
- Is the code clean and maintainable?
|
|
57
|
+
|
|
58
|
+
**Discipline:**
|
|
59
|
+
- Did I avoid overbuilding (YAGNI)?
|
|
60
|
+
- Did I only build what was requested?
|
|
61
|
+
- Did I follow existing patterns in the codebase?
|
|
62
|
+
|
|
63
|
+
**Testing:**
|
|
64
|
+
- Do tests actually verify behavior (not just mock behavior)?
|
|
65
|
+
- Did I follow TDD if required?
|
|
66
|
+
- Are tests comprehensive?
|
|
67
|
+
|
|
68
|
+
If you find issues during self-review, fix them now before reporting.
|
|
69
|
+
|
|
70
|
+
## Report Format
|
|
71
|
+
|
|
72
|
+
When done, report:
|
|
73
|
+
- What you implemented
|
|
74
|
+
- What you tested and test results
|
|
75
|
+
- Files changed
|
|
76
|
+
- Self-review findings (if any)
|
|
77
|
+
- Any issues or concerns
|
|
78
|
+
```
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
# Spec Compliance Reviewer Prompt Template
|
|
2
|
+
|
|
3
|
+
Use this template when running a spec compliance review step in single-flow mode.
|
|
4
|
+
|
|
5
|
+
**Purpose:** Verify implementer built what was requested (nothing more, nothing less)
|
|
6
|
+
|
|
7
|
+
```
|
|
8
|
+
task_boundary:
|
|
9
|
+
description: "Review spec compliance for Task N"
|
|
10
|
+
prompt: |
|
|
11
|
+
You are reviewing whether an implementation matches its specification.
|
|
12
|
+
|
|
13
|
+
## What Was Requested
|
|
14
|
+
|
|
15
|
+
[FULL TEXT of task requirements]
|
|
16
|
+
|
|
17
|
+
## What Implementer Claims They Built
|
|
18
|
+
|
|
19
|
+
[From implementer's report]
|
|
20
|
+
|
|
21
|
+
## CRITICAL: Do Not Trust the Report
|
|
22
|
+
|
|
23
|
+
The implementer finished suspiciously quickly. Their report may be incomplete,
|
|
24
|
+
inaccurate, or optimistic. You MUST verify everything independently.
|
|
25
|
+
|
|
26
|
+
**DO NOT:**
|
|
27
|
+
- Take their word for what they implemented
|
|
28
|
+
- Trust their claims about completeness
|
|
29
|
+
- Accept their interpretation of requirements
|
|
30
|
+
|
|
31
|
+
**DO:**
|
|
32
|
+
- Read the actual code they wrote
|
|
33
|
+
- Compare actual implementation to requirements line by line
|
|
34
|
+
- Check for missing pieces they claimed to implement
|
|
35
|
+
- Look for extra features they didn't mention
|
|
36
|
+
|
|
37
|
+
## Your Job
|
|
38
|
+
|
|
39
|
+
Read the implementation code and verify:
|
|
40
|
+
|
|
41
|
+
**Missing requirements:**
|
|
42
|
+
- Did they implement everything that was requested?
|
|
43
|
+
- Are there requirements they skipped or missed?
|
|
44
|
+
- Did they claim something works but didn't actually implement it?
|
|
45
|
+
|
|
46
|
+
**Extra/unneeded work:**
|
|
47
|
+
- Did they build things that weren't requested?
|
|
48
|
+
- Did they over-engineer or add unnecessary features?
|
|
49
|
+
- Did they add "nice to haves" that weren't in spec?
|
|
50
|
+
|
|
51
|
+
**Misunderstandings:**
|
|
52
|
+
- Did they interpret requirements differently than intended?
|
|
53
|
+
- Did they solve the wrong problem?
|
|
54
|
+
- Did they implement the right feature but wrong way?
|
|
55
|
+
|
|
56
|
+
**Verify by reading code, not by trusting report.**
|
|
57
|
+
|
|
58
|
+
Report:
|
|
59
|
+
- ✅ Spec compliant (if everything matches after code inspection)
|
|
60
|
+
- ❌ Issues found: [list specifically what's missing or extra, with file:line references]
|
|
61
|
+
```
|
|
@@ -0,0 +1,44 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: skill-creator
|
|
3
|
+
description: "Chuyên gia tạo mới và chỉnh sửa Antigravity Skills. ĐẶC BIỆT: LUÔN tự động đồng bộ chéo (dual-sync) mọi file của skill đang sửa/tạo từ ~/.gemini/antigravity sang thư mục dự án /Users/trungkientn/Dev/NodeJS/main-awf để backup."
|
|
4
|
+
version: 1.0.0
|
|
5
|
+
trigger: conditional
|
|
6
|
+
activation_keywords:
|
|
7
|
+
- "cập nhật skill"
|
|
8
|
+
- "tạo skill mới"
|
|
9
|
+
- "viết skill"
|
|
10
|
+
- "chỉnh sửa skill"
|
|
11
|
+
- "bổ sung skill"
|
|
12
|
+
---
|
|
13
|
+
|
|
14
|
+
# 🛠️ Skill Creator (with Auto-Sync to main-awf)
|
|
15
|
+
|
|
16
|
+
**Purpose**: Hỗ trợ người dùng thiết kế, lập trình và tinh chỉnh các Antigravity Skills theo chuẩn.
|
|
17
|
+
**Core Rule**: Mọi tác vụ trên skill bất kỳ (tạo mới, thay đổi nội dung thư mục) BẮT BUỘC phải được nhân bản tự động sang repo `main-awf` ngay lập tức để giữ đồng bộ phiên bản mà không cần user nhắc nhở.
|
|
18
|
+
|
|
19
|
+
## 🚀 Luồng tương tác (Workflow)
|
|
20
|
+
|
|
21
|
+
### Bước 1: Thu thập yêu cầu (Requirements)
|
|
22
|
+
- Nắm bắt thông tin về Skill user muốn tạo/sửa: Tên skill, công dụng cốt lõi, từ khoá kích hoạt (triggers), và các script chuyên biệt (Python, Bash, Node) nếu có.
|
|
23
|
+
- Nếu là skill mới, định hình các thư mục cần thiết (vd: `scripts/`, `templates/`, `resources/`).
|
|
24
|
+
|
|
25
|
+
### Bước 2: Thiết kế và Lập trình (Implementation)
|
|
26
|
+
- Tạo thư mục gốc của skill tại `~/.gemini/antigravity/skills/<skill-name>`.
|
|
27
|
+
- Khởi tạo (hoặc cập nhật) file điều hướng cốt lõi `SKILL.md` tuân thủ YAML frontmatter.
|
|
28
|
+
- Lập trình các script logic chuyên biệt và đặt đúng vào các thư mục tương ứng bên trong.
|
|
29
|
+
|
|
30
|
+
### Bước 3: ĐỒNG BỘ HOÁ BẮT BUỘC (The Mandatory Sync)
|
|
31
|
+
Ngay sau khi thao tác sinh file nội bộ thư mục `.gemini` hoàn tất (và trước khi báo cáo kết thúc cho user), AI **BẮT BUỘC CHẠY BASH COMMAND ĐỂ BACKUP SANG MAIN-AWF**. Đây là nguyên tắc sống còn của skill này:
|
|
32
|
+
|
|
33
|
+
```bash
|
|
34
|
+
# 1. Khởi tạo thư mục đích tại main-awf để tránh lỗi NotFound
|
|
35
|
+
mkdir -p /Users/trungkientn/Dev/NodeJS/main-awf/skills/<skill-name>
|
|
36
|
+
|
|
37
|
+
# 2. Quét và chép đè mọi thứ thuộc skill từ Antigravity nội bộ sang repo Node
|
|
38
|
+
cp -R ~/.gemini/antigravity/skills/<skill-name>/* /Users/trungkientn/Dev/NodeJS/main-awf/skills/<skill-name>/
|
|
39
|
+
```
|
|
40
|
+
*(AI chú ý: Nhớ thay chữ `<skill-name>` bằng đúng tên thư mục skill đang xử lý. Không được bỏ sót thư mục con như scripts).*
|
|
41
|
+
|
|
42
|
+
### Bước 4: Nghiệm thu (Review)
|
|
43
|
+
- Báo cáo cho người dùng tình trạng tạo/sửa skill.
|
|
44
|
+
- Xác nhận bằng định dạng Check-box rằng: `✅ Đã đồng bộ an toàn sang repo main-awf`.
|