feat: unify cloud resource management with .env integration - #100
Conversation
|
Hi @mouradsme, Thank you for this contribution — and for being the first contributor to CodFlow! 🎉 I reviewed the PR thoroughly (full read of every file, plus testing the What we verified and like:
Before we can merge, there are three blockers we'd ask you to address: 1. Rebase onto current We merged a large PR (#102) after you branched, and two files now conflict: 2. Fail loudly when the deploy URL is missing (silent localhost deploy) In const { serverUrl } = getCloudEnv();
if (serverUrl === "http://localhost:8787") {
console.error(
"COD_SERVER_URL resolved to the localhost default.\n" +
"Set COD_SERVER_URL in the root .env (see .env.example) before deploying,\n" +
"or pass --force-local to deploy anyway."
);
process.exit(1);
}3. The KV namespace keys are dead config
Smaller items (nice-to-have, not blocking):
Once the rebase and the deploy guard are in, we'll re-run the full test |
…t enhancements # Conflicts: # cod-astro/theme01/package.json # cod-astro/theme01/wrangler.jsonc
253968a to
795e060
Compare
docs: align setup skill, docs and security suite with unified .env cloud config (#100 follow-up)
Unified CodFlow Cloud resource values — single source of truth for the seeder scripts, the D1 migration wrapper, the R2 CORS setup, and the storefront deploy helper.