From 546e9d661734358dfd3d593d4667246874589880 Mon Sep 17 00:00:00 2001 From: danielshih Date: Tue, 21 Oct 2025 21:18:57 +0800 Subject: [PATCH 1/3] Support more lang/Devops of Common Patterns by Technology - lang: TypeScript, Swift, R. - Devops: Kubernetes/k8s --- templates/commands/implement.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/templates/commands/implement.md b/templates/commands/implement.md index c4e7c17b..0b27c532 100644 --- a/templates/commands/implement.md +++ b/templates/commands/implement.md @@ -74,7 +74,7 @@ You **MUST** consider the user input before proceeding (if not empty). **If ignore file missing**: Create with full pattern set for detected technology **Common Patterns by Technology** (from plan.md tech stack): - - **Node.js/JavaScript**: `node_modules/`, `dist/`, `build/`, `*.log`, `.env*` + - **Node.js/JavaScript/TypeScript**: `node_modules/`, `dist/`, `build/`, `*.log`, `.env*` - **Python**: `__pycache__/`, `*.pyc`, `.venv/`, `venv/`, `dist/`, `*.egg-info/` - **Java**: `target/`, `*.class`, `*.jar`, `.gradle/`, `build/` - **C#/.NET**: `bin/`, `obj/`, `*.user`, `*.suo`, `packages/` @@ -85,6 +85,8 @@ You **MUST** consider the user input before proceeding (if not empty). - **Kotlin**: `build/`, `out/`, `.gradle/`, `.idea/`, `*.class`, `*.jar`, `*.iml`, `*.log`, `.env*` - **C++**: `build/`, `bin/`, `obj/`, `out/`, `*.o`, `*.so`, `*.a`, `*.exe`, `*.dll`, `.idea/`, `*.log`, `.env*` - **C**: `build/`, `bin/`, `obj/`, `out/`, `*.o`, `*.a`, `*.so`, `*.exe`, `Makefile`, `config.log`, `.idea/`, `*.log`, `.env*` + - **Swift**: `.build/`, `DerivedData/`, `*.xcodeproj/`, `*.xcworkspace/`, `*.swiftpm/`, `Packages/` + - **R**: `.Rproj.user/`, `.Rhistory`, `.RData`, `.Ruserdata`, `*.Rproj`, `packrat/`, `renv/` - **Universal**: `.DS_Store`, `Thumbs.db`, `*.tmp`, `*.swp`, `.vscode/`, `.idea/` **Tool-Specific Patterns**: @@ -92,6 +94,7 @@ You **MUST** consider the user input before proceeding (if not empty). - **ESLint**: `node_modules/`, `dist/`, `build/`, `coverage/`, `*.min.js` - **Prettier**: `node_modules/`, `dist/`, `build/`, `coverage/`, `package-lock.json`, `yarn.lock`, `pnpm-lock.yaml` - **Terraform**: `.terraform/`, `*.tfstate*`, `*.tfvars`, `.terraform.lock.hcl` + - **Kubernetes/k8s**: `*.secret.yaml`, `secrets/`, `.kube/`, `kubeconfig*`, `*.key`, `*.crt` 5. Parse tasks.md structure and extract: - **Task phases**: Setup, Tests, Core, Integration, Polish From 02c1549f8015d94966c6dd507ecc0c3e4e297300 Mon Sep 17 00:00:00 2001 From: danielshih Date: Tue, 21 Oct 2025 21:27:46 +0800 Subject: [PATCH 2/3] fix regarding copilot suggestion --- templates/commands/implement.md | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/templates/commands/implement.md b/templates/commands/implement.md index 0b27c532..9a804a0b 100644 --- a/templates/commands/implement.md +++ b/templates/commands/implement.md @@ -34,14 +34,14 @@ You **MUST** consider the user input before proceeding (if not empty). - Calculate overall status: * **PASS**: All checklists have 0 incomplete items * **FAIL**: One or more checklists have incomplete items - + - **If any checklist is incomplete**: * Display the table with incomplete item counts * **STOP** and ask: "Some checklists are incomplete. Do you want to proceed with implementation anyway? (yes/no)" * Wait for user response before continuing * If user says "no" or "wait" or "stop", halt execution * If user says "yes" or "proceed" or "continue", proceed to step 3 - + - **If all checklists are complete**: * Display the table showing all checklists passed * Automatically proceed to step 3 @@ -56,7 +56,7 @@ You **MUST** consider the user input before proceeding (if not empty). 4. **Project Setup Verification**: - **REQUIRED**: Create/verify ignore files based on actual project setup: - + **Detection & Creation Logic**: - Check if the following command succeeds to determine if the repository is a git repo (create/verify .gitignore if so): @@ -69,10 +69,10 @@ You **MUST** consider the user input before proceeding (if not empty). - Check if .npmrc or package.json exists → create/verify .npmignore (if publishing) - Check if terraform files (*.tf) exist → create/verify .terraformignore - Check if .helmignore needed (helm charts present) → create/verify .helmignore - + **If ignore file already exists**: Verify it contains essential patterns, append missing critical patterns only **If ignore file missing**: Create with full pattern set for detected technology - + **Common Patterns by Technology** (from plan.md tech stack): - **Node.js/JavaScript/TypeScript**: `node_modules/`, `dist/`, `build/`, `*.log`, `.env*` - **Python**: `__pycache__/`, `*.pyc`, `.venv/`, `venv/`, `dist/`, `*.egg-info/` @@ -85,10 +85,10 @@ You **MUST** consider the user input before proceeding (if not empty). - **Kotlin**: `build/`, `out/`, `.gradle/`, `.idea/`, `*.class`, `*.jar`, `*.iml`, `*.log`, `.env*` - **C++**: `build/`, `bin/`, `obj/`, `out/`, `*.o`, `*.so`, `*.a`, `*.exe`, `*.dll`, `.idea/`, `*.log`, `.env*` - **C**: `build/`, `bin/`, `obj/`, `out/`, `*.o`, `*.a`, `*.so`, `*.exe`, `Makefile`, `config.log`, `.idea/`, `*.log`, `.env*` - - **Swift**: `.build/`, `DerivedData/`, `*.xcodeproj/`, `*.xcworkspace/`, `*.swiftpm/`, `Packages/` + - **Swift**: `.build/`, `DerivedData/`, `*.swiftpm/`, `Packages/` - **R**: `.Rproj.user/`, `.Rhistory`, `.RData`, `.Ruserdata`, `*.Rproj`, `packrat/`, `renv/` - **Universal**: `.DS_Store`, `Thumbs.db`, `*.tmp`, `*.swp`, `.vscode/`, `.idea/` - + **Tool-Specific Patterns**: - **Docker**: `node_modules/`, `.git/`, `Dockerfile*`, `.dockerignore`, `*.log*`, `.env*`, `coverage/` - **ESLint**: `node_modules/`, `dist/`, `build/`, `coverage/`, `*.min.js` @@ -104,7 +104,7 @@ You **MUST** consider the user input before proceeding (if not empty). 6. Execute implementation following the task plan: - **Phase-by-phase execution**: Complete each phase before moving to the next - - **Respect dependencies**: Run sequential tasks in order, parallel tasks [P] can run together + - **Respect dependencies**: Run sequential tasks in order, parallel tasks [P] can run together - **Follow TDD approach**: Execute test tasks before their corresponding implementation tasks - **File-based coordination**: Tasks affecting the same files must run sequentially - **Validation checkpoints**: Verify each phase completion before proceeding From e0e62f67579825239361f9256baf85ad233aa313 Mon Sep 17 00:00:00 2001 From: danielshih Date: Tue, 21 Oct 2025 21:30:47 +0800 Subject: [PATCH 3/3] revert vscode auto remove extra space --- templates/commands/implement.md | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/templates/commands/implement.md b/templates/commands/implement.md index 9a804a0b..8e19caf9 100644 --- a/templates/commands/implement.md +++ b/templates/commands/implement.md @@ -34,14 +34,14 @@ You **MUST** consider the user input before proceeding (if not empty). - Calculate overall status: * **PASS**: All checklists have 0 incomplete items * **FAIL**: One or more checklists have incomplete items - + - **If any checklist is incomplete**: * Display the table with incomplete item counts * **STOP** and ask: "Some checklists are incomplete. Do you want to proceed with implementation anyway? (yes/no)" * Wait for user response before continuing * If user says "no" or "wait" or "stop", halt execution * If user says "yes" or "proceed" or "continue", proceed to step 3 - + - **If all checklists are complete**: * Display the table showing all checklists passed * Automatically proceed to step 3 @@ -56,7 +56,7 @@ You **MUST** consider the user input before proceeding (if not empty). 4. **Project Setup Verification**: - **REQUIRED**: Create/verify ignore files based on actual project setup: - + **Detection & Creation Logic**: - Check if the following command succeeds to determine if the repository is a git repo (create/verify .gitignore if so): @@ -69,10 +69,10 @@ You **MUST** consider the user input before proceeding (if not empty). - Check if .npmrc or package.json exists → create/verify .npmignore (if publishing) - Check if terraform files (*.tf) exist → create/verify .terraformignore - Check if .helmignore needed (helm charts present) → create/verify .helmignore - + **If ignore file already exists**: Verify it contains essential patterns, append missing critical patterns only **If ignore file missing**: Create with full pattern set for detected technology - + **Common Patterns by Technology** (from plan.md tech stack): - **Node.js/JavaScript/TypeScript**: `node_modules/`, `dist/`, `build/`, `*.log`, `.env*` - **Python**: `__pycache__/`, `*.pyc`, `.venv/`, `venv/`, `dist/`, `*.egg-info/` @@ -88,7 +88,7 @@ You **MUST** consider the user input before proceeding (if not empty). - **Swift**: `.build/`, `DerivedData/`, `*.swiftpm/`, `Packages/` - **R**: `.Rproj.user/`, `.Rhistory`, `.RData`, `.Ruserdata`, `*.Rproj`, `packrat/`, `renv/` - **Universal**: `.DS_Store`, `Thumbs.db`, `*.tmp`, `*.swp`, `.vscode/`, `.idea/` - + **Tool-Specific Patterns**: - **Docker**: `node_modules/`, `.git/`, `Dockerfile*`, `.dockerignore`, `*.log*`, `.env*`, `coverage/` - **ESLint**: `node_modules/`, `dist/`, `build/`, `coverage/`, `*.min.js` @@ -104,7 +104,7 @@ You **MUST** consider the user input before proceeding (if not empty). 6. Execute implementation following the task plan: - **Phase-by-phase execution**: Complete each phase before moving to the next - - **Respect dependencies**: Run sequential tasks in order, parallel tasks [P] can run together + - **Respect dependencies**: Run sequential tasks in order, parallel tasks [P] can run together - **Follow TDD approach**: Execute test tasks before their corresponding implementation tasks - **File-based coordination**: Tasks affecting the same files must run sequentially - **Validation checkpoints**: Verify each phase completion before proceeding