Conversation
…nv contract test check-envs.sh in nest-server-starter could never pass: it waited for "Server startet at" while the app logs "Server started at", and its Phase 2 copied tests/fixtures/.env.deployed-test, which .gitignore keeps out of every clone. fullstack-updater and modernizing-toolchain still required its "All env configurations OK." as a done signal, and the migration checklist prescribed a CI job around it. Acceptance is now src/config.env.spec.ts covering REQUIRED_DEPLOYED_ENV_VARS and passing in `<pm> test`, matching nest-server-starter PR #57. Phase 8 describes that test and tells projects to delete a legacy check-envs.sh. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
fullstack-updaterandmodernizing-toolchainrequired<pm> run check:envsto print "All env configurations OK." as a done signal. That could never happen:scripts/check-envs.shin nest-server-starter was broken on every machine.Measured in nest-server-starter at
c3912e8:scripts/check-envs.sh:137waited forServer startet at, butsrc/main.ts:140logsServer started at(the typo came in with00bf0d8). All nine boot cases hit the 60s timeout.scripts/check-envs.sh:16copiedtests/fixtures/.env.deployed-test, which.gitignore:82(.env.*) kept out of every clone, so Phase 2 aborted underset -e.scripts/check.mjsnor.github/workflows/test.ymlrunscheck:envs, so nothing ever caught this.nest-server-starter #57 (merged as
e66263c) removed the script and moved the check intosrc/config.env.spec.ts, so it now runs in everypnpm test.What changes
fullstack-updatersuccess criteria,modernizing-toolchaindone signals):src/config.env.spec.tscovers every entry ofREQUIRED_DEPLOYED_ENV_VARSand passes in<pm> test.check-envs.shplus a fixture. It keeps a dated incident note so the script does not get reintroduced.check:envsentries are removed from the Phase 5 script blocks. Thecheck:envsjob is removed from the Phase 10 GitLab CI recipe, andcheck-envs.shis removed from the upstream file list inchecking-upstream-first.Checked
claude plugin validate plugins/lt-devpasses,scripts/scan-secrets.shis clean, andcheck-cross-references.tsreports 0 broken references.run <script>name and everyscripts/*path inplugins/lt-devagainst the local clones of both starters, lt-monorepo and the CLI. No other dead targets turned up.Not checked / not changed
checkchain in the migration checklist still shows the old inline chain, while the starter now usesnode scripts/check.mjs. That is drift, not an unpassable criterion, so it is left for a separate change.🤖 Generated with Claude Code