Resolve stash conflicts: keep upstream + local deploy wiring checks
Some checks are pending
CI / pr-title (push) Waiting to run
CI / test (push) Waiting to run
Release PR / release-pr (push) Waiting to run
Release PR / publish-cli (push) Blocked by required conditions

Merge upstream's ce-brainstorm skip-menu guidance and ce-plan
repo-research-analyst integration with local deploy wiring flag
additions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
John Lamb
2026-03-25 13:35:02 -05:00
parent 8279c8ddc3
commit 6695dd35f7
4 changed files with 5 additions and 5 deletions

View File

@@ -144,6 +144,8 @@ For each approach, provide:
Lead with your recommendation and explain why. Prefer simpler solutions when added complexity creates real carrying cost, but do not reject low-cost, high-value polish just because it is not strictly necessary.
**Deploy wiring flag:** If any approach introduces new backend env vars or config fields, call this out explicitly in the approach description. Deploy values files (e.g. `values.yaml`, `.env.*`, Terraform vars) must be updated alongside the config code — not as a follow-up. This is a hard-won lesson; see `docs/solutions/deployment-issues/missing-env-vars-in-values-yaml.md`.
If one approach is clearly best and alternatives are not meaningful, skip the menu and state the recommendation directly.
If relevant, call out whether the choice is: