docs: align setup skill, docs and security suite with unified .env cloud config (#100 follow-up) - #106
Merged
Merged
Conversation
…oud config (#100 follow-up) - setup skill: COD_SERVER_URL now set in root .env (not theme01 wrangler.jsonc), note the loopback deploy guard, sync .agents copy with .opencode (Sendili section) - security suite (common.sh): DB_NAME resolves via cloud-env.mjs, DB_NAME env override kept - DEPLOYMENT/CONFIGURATION/theme01 AGENTS: vars block is gone, deploy is .env-driven - de-hardcode codflow-os-db in README, CONTRIBUTING, EMAIL-SENDING, WHATSAPP-OTP docs
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
1 participant
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.
Follow-up to #100 (merged). The code was verified and merged; this closes the docs/config drift it left behind.
What
.agentsand.opencodecopies): removed stale instructions to setCOD_SERVER_URLincod-astro/theme01/wrangler.jsonc(that vars block no longer exists); documented thatCOD_SERVER_URLis set in the root..envand thatnpm run deployrefuses a localhost value without--force-local. Synced the.agentscopy, which was also missing the Step 6b Sendili section.cod-server/scripts/security/lib/common.sh:DB_NAMEnow resolves viacod-server/scripts/cloud-env.mjs(root.envCOD_DB_NAME), keeping theDB_NAME=<name>env override and a safe fallback — this was the one remaining script hardcode and the reason the setup skill's grep gate failed.docs/DEPLOYMENT.md,docs/CONFIGURATION.md,cod-astro/theme01/AGENTS.md: replaced instructions pointing at the removedvarsblock with the new.env-driven deploy flow.codflow-os-dbin example commands:cod-server/README.md,CONTRIBUTING.md,docs/EMAIL-SENDING.md,docs/WHATSAPP-OTP-VERIFICATION.md,.agents/skills/whatsapp-otp/PLAN.md— commands now referenceCOD_DB_NAME/npm run db:migrate:remote/node scripts/d1.mjs.Verification
common.sh:bash -n+ sourced resolution check (DB_NAME=codflow-os-dbfrom.env, override works)codflow-os-dbleft in scripts/docs (only by-design defaults incloud-env.mjs,.env.example, wrangler templates)Docs-only change except
common.sh, which is behavior-compatible (same default, now overridable via.env).