fix(cli): persist auto-scale rules in JSON mode - #997
Conversation
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
6 similar comments
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
What
--dry-runis absent--json --dry-runWhy
sh1pt scale auto --jsonreturned the requested rules and exited beforesaveAutoScaleRules(). Automation saw a successful JSON response even though no configuration file existed, so the requested auto-scale policy was silently discarded.Reproduction
Before this change, the command printed valid JSON but
.sh1pt/auto-scale.jsondid not exist. After the change, the same command writes the file; adding--dry-runstill skips the write.Testing
pnpm exec vitest run packages/cli/src/commands/scale.test.ts(55 passed)pnpm --filter @profullstack/sh1pt... buildpnpm --filter @profullstack/sh1pt typecheck