Skip to content

[codex] chore: consolidate env config#1341

Closed
Zetazzz wants to merge 1 commit into
constructive-io:mainfrom
Zetazzz:chore/env-config-consolidation-main
Closed

[codex] chore: consolidate env config#1341
Zetazzz wants to merge 1 commit into
constructive-io:mainfrom
Zetazzz:chore/env-config-consolidation-main

Conversation

@Zetazzz

@Zetazzz Zetazzz commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Consolidates server-related environment configuration into the existing PGPM/Constructive options pipeline.

Changes

  • Adds captcha.recaptchaSecretKey and upload.maxFileSize to PGPM and Constructive option types.
  • Parses RECAPTCHA_SECRET_KEY and MAX_UPLOAD_FILE_SIZE in @pgpmjs/env.
  • Passes CAPTCHA and GraphQL upload configuration into graphql-server from effectiveOpts instead of reading env directly in middleware/runtime setup.
  • Adds CAPTCHA middleware coverage proving the secret comes from explicit config rather than direct process.env access.
  • Updates env merge tests, snapshots, and env README documentation.

Validation

  • pnpm --filter @pgpmjs/types build
  • pnpm --filter @pgpmjs/env test
  • pnpm --filter @pgpmjs/env build
  • pnpm --filter @constructive-io/graphql-types build
  • pnpm --filter @constructive-io/graphql-env test
  • pnpm --filter @constructive-io/graphql-env build
  • pnpm --filter @constructive-io/express-context build
  • pnpm --filter @constructive-io/graphql-server test -- captcha.test.ts
  • pnpm --filter @constructive-io/graphql-server build
  • git diff --cached --check

@Zetazzz

Zetazzz commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

Closing in favor of same-repository PR #1342 from constructive-io/constructive:chore/env-config-consolidation-main.

@Zetazzz Zetazzz closed this Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant