Skip to content

Comments

dev branch configuration#1947

Open
lolopinto wants to merge 10 commits intomainfrom
ent-dev-branches-config
Open

dev branch configuration#1947
lolopinto wants to merge 10 commits intomainfrom
ent-dev-branches-config

Conversation

@lolopinto
Copy link
Owner

@lolopinto lolopinto commented Jan 25, 2026

Summary

  • Enforce dev schema isolation defaults (includePublic=false, search_path dev-only) and consistent enablement rules across Go/TS.
  • Add ignoreBranches (e.g. main/master) to skip dev schemas unless ENT_DEV_SCHEMA_ENABLED=true.
  • Sanitize explicit schema names (lowercase, non-alnum -> _, max 63 chars, prefix with schema_ if leading digit).
  • Propagate dev schema options through codegen/state, TS runtime, and auto_schema; docs/AGENTS updated.
  • Make registry touches best-effort and remove runtime DDL in TS.
  • Update docker changelog for dev schema config/pruning work (auto_schema changes for making changes in other schemas not just public #1946, dev branch configuration #1947).

addresses #499

Behavior notes

  • devSchema.enabled is authoritative; schemaName alone does not enable.
  • ignoreBranches applies only when not force-enabled via ENT_DEV_SCHEMA_ENABLED.
  • Branch mismatch checks apply when using the state file (not explicit runtime config).

Tests

  • go test ./internal/devschema
  • jest src/core/dev_schema.test.ts

@lolopinto lolopinto marked this pull request as ready for review January 26, 2026 05:26
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