Skip to content

Commit 9d91546

Browse files
fix(skills/update-stack): add billing to stack module list + broaden test exclusion
- Add `billing` to stack module declaration (it exists in devkit/Node: modules/billing/) — gate and header were inconsistent - Update conflict resolution table accordingly - Broaden test-file exclusion: add /__tests__/ directory pattern and extend extensions to (js|jsx|ts|tsx) for consistency Per CodeRabbit review on #3760
1 parent 3829536 commit 9d91546

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

.claude/skills/update-stack/SKILL.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ Two-phase workflow. Phase 1 brings the stack down ISO. Phase 2 aligns the projec
1717

1818
**Goal: stack modules and lib exit this phase identical to upstream. Zero downstream logic in them.**
1919

20-
Stack modules: `home`, `auth`, `users`, `tasks`, `uploads` — Stack core: `lib/` (existing files), `config/defaults/` (stack-owned files only)
20+
Stack modules: `home`, `auth`, `users`, `tasks`, `uploads`, `billing` — Stack core: `lib/` (existing files), `config/defaults/` (stack-owned files only)
2121

2222
### 1. Setup remote + merge
2323

@@ -31,7 +31,7 @@ git merge devkit-node/master
3131

3232
| File | Rule |
3333
|------|------|
34-
| Stack module (`modules/home\|auth\|users\|tasks\|uploads`) | `git checkout --theirs <file>` |
34+
| Stack module (`modules/home\|auth\|users\|tasks\|uploads\|billing`) | `git checkout --theirs <file>` |
3535
| `lib/<existing-file>` | `git checkout --theirs <file>` (existing stack framework files — always ISO) |
3636
| `config/defaults/development.js`, `production.js`, etc. | `git checkout --theirs <file>` (stack-owned defaults) |
3737
| `package-lock.json` | `git checkout --theirs package-lock.json` — regenerate after `package.json` is resolved |
@@ -109,7 +109,7 @@ while IFS= read -r f; do
109109
fi
110110
fi
111111
done < <(git ls-files modules/home modules/auth modules/users modules/tasks modules/uploads modules/billing lib config/defaults 2>/dev/null \
112-
| grep -v "/tests/" | grep -vE "\.(test|spec)\.js$")
112+
| grep -vE "/(tests|__tests__)/" | grep -vE "\.(test|spec)\.(js|jsx|ts|tsx)$")
113113

114114
[ "$drift_found" -eq 1 ] && exit 1
115115
echo "3ter: no undeclared drift — OK"

0 commit comments

Comments
 (0)