Skip to content

feat(state): make production authority explicit - #193

Draft
seonghobae wants to merge 14 commits into
feat/runtime-config-bootstrap-snapshotfrom
codex/postgres-state-authority-mode
Draft

feat(state): make production authority explicit#193
seonghobae wants to merge 14 commits into
feat/runtime-config-bootstrap-snapshotfrom
codex/postgres-state-authority-mode

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Refs #80 #192. Draft child of current #140 Runtime Configuration foundation.

Bounded production-authority contract

Wardnet must never infer deployment intent from listener topology: production may bind loopback behind a proxy/sidecar. This slice makes mutable-state authority explicit while leaving the actual PostgreSQL repository/migrations/RLS/recovery under #80/#192.

  • production requires explicit PostgreSQL authority;
  • file authority requires a non-empty path;
  • dormant file paths beside memory/PostgreSQL authority are rejected;
  • standalone preserves compatible memory/file behavior;
  • selecting PostgreSQL remains fail closed before listener startup until the durable adapter is integrated, so an explicit production declaration cannot silently downgrade to memory/file state.

It does not claim PostgreSQL durability, RLS, tenant isolation, migration/recovery, backup/restore, source-generation uniqueness or release readiness.

Hostile RED → causal repair lineage

Test-only 37f5c8cb73ed45a003e1d4ca69771d649db5a5f7 produced the original hosted semantic RED in CI 34175687088 / rust 101904485782: checkout/toolchain/formatting passed and locked workspace tests failed because the explicit deployment/state-authority contract did not exist. The minimum implementation then reached source GREEN.

Fresh review found a second fail-closed edge: StateAuthority::File rejected None but accepted PathBuf::from(""). Test-only exact 2d57983d38a8e712a6103f6898906ed88b569be0, CI 34177340458 / rust 101909271259, passed checkout/toolchain/formatting and failed semantically on the empty-path case. Repair f50e7204df5e6467297969eddff70891358e4bff made tests pass but strict Clippy rejected the spelling; exact 88e54cd18fb686a6b61525c400531820f7a014b4 applied only Clippy's semantics-preserving Option::is_none_or form.

Current parent adoption and exact-current evidence — 2026-09-09 KST

#140 subsequently advanced through additional hostile Runtime Configuration environment-alias fitness repairs to exact 93a51f9706cf8a9704f69aed4a69df5be16c84e4. This branch adopted that intervening foundation normally rather than treating it as a race or copying it.

Current exact head is 7a93322c9a824e6e939a8143b1f20eccaeac856d, commit chore(stack): adopt current #140 foundation into #193. GitHub records base exact #140@93a51f9706cf8a9704f69aed4a69df5be16c84e4, Draft and mechanically mergeable. Fresh comparison from the prior #193 GREEN 88e54cd... to current head is ahead 8 / behind 0 and the intervening effective delta is confined to src/runtime_config.rs, i.e. the parent foundation movement rather than a second state-authority implementation.

On unchanged exact 7a93322c9a824e6e939a8143b1f20eccaeac856d:

  • CI 34234733846 — terminal SUCCESS;
  • Fuzz 34234733840 — terminal SUCCESS;
  • submitted reviews — 0;
  • inline review threads — 0.

The earlier 88e54cd... checks are predecessor evidence only; current exact executions above are the bounded child evidence. The branch remains Draft because #140 is still unintegrated protected truth and all later PostgreSQL children depend on this exact stack ancestry.

Integration boundary

Keep order #140 -> #193 -> #194 -> #196 -> #198 -> #199 -> #200 -> #207 -> #208 -> #209 -> #212 -> #216 -> #217 -> #219 -> #221 -> #223 -> #224 -> #225 -> #226 -> #228 -> #229 -> #231 -> #233. Any parent movement is adopted non-force and invalidates dependent gate evidence.

No force update, destructive rebase, self/model approval, routine administrator bypass, gate weakening, mutable foreign dependency, source copy, cross-service SQL, no-op dispatch churn or predecessor-evidence transfer.

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Ordinary non-force stack repair. Preserve #193 state-authority delta and adopt #140 runtime-env alias fitness repairs.
seonghobae added a commit that referenced this pull request Sep 8, 2026
Ordinary non-force restack; preserve the #194 credential boundary and adopt the current parent ancestry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request priority: medium Normal-priority or P2 work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant