feat(integration): add concrete PostgreSQL plugin runtime - #245
Draft
seonghobae wants to merge 72 commits into
Draft
feat(integration): add concrete PostgreSQL plugin runtime#245seonghobae wants to merge 72 commits into
seonghobae wants to merge 72 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This was referenced Sep 4, 2026
20 tasks
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
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.
Buyer/security outcome
Advances #130/#244 from an abstract hosted PostgreSQL seam to an Integration-owned node-postgres/default-entrypoint implementation without borrowing another bounded context's Pool, using generic
DATABASE_URL, or treating process-wide PostgreSQL settings as authority.Current non-force ancestry
Parent #244 remains
f0b91869f306b1b7d0d5759307acf347e413aa06. Current exact head0a6fe7b25664bc893a4ebeb9647d830b59202fc2is a non-force descendant of prior accepted head45b5ad55b1575a677c9edf65fc350e55e3c68f8fand preserves the Integration-owned ten-path product delta. The protected #218 runner/control-plane foundation remains normal ancestry; no predecessor check/review evidence transfers.Latest RED → repair status
RED
a0b5653426f0d55e50f6baff6ad66543a57d932fremains valid:apps/integration-service/package.jsondeclarespg ^8.22.0and@types/pg ^8.20.0, while rootpnpm-lock.yamlstill lacks the two direct entries under theapps/integration-serviceimporter. The shared lock already resolvespg 8.22.0and@types/pg 8.20.3.An attempted full-file contents mutation at
c78de4f61a8fdfbeaa11d9bba09d64391edce0f6was rejected as invalid because it did not preserve the canonical lockfile. It was immediately repaired by non-force descendant0a6fe7b25664bc893a4ebeb9647d830b59202fc2, whose tree restorespnpm-lock.yamlto the exact prior blobac72584e4cc0a7307aee08b15a71f468eb7cdba2. That invalid mutation is not accepted evidence and did not weaken or replace the frozen-lock RED.The accepted next repair is still importer-only: add only
pg { specifier: ^8.22.0, version: 8.22.0 }and@types/pg { specifier: ^8.20.0, version: 8.20.3 }underapps/integration-service, with no package graph/integrity/CLI churn. Earlier broad lock regenerationf3bc29112e691e1c30008c4e4132b486f20567ffchanged unrelated lockfile data and was likewise rejected/reverted byb6c1befd8fea6b9943d95461b9f19b1d9f4f5061.Earlier preserved lineage covers credential-free process startup failure; explicit self-contained PostgreSQL URI authority without
PG*fallback; URI query/fragment denial; separately configured verified TLS; finite connection/statement/query/idle/lifetime bounds; bounded idle-error SQLSTATE evidence and telemetry failure; constructor/listener acquisition cleanup; lazy-Pool readiness before runtime authority; untrusted readiness-result access; one-time query-capability capture; and one-time shutdown-capability capture so readiness, accepted runtime SQL and cleanup retain the acquired authority.Evidence boundary
The current exact head has source/fixture/architecture/changelog evidence only. Do not claim package/repository/100% coverage GREEN, real PostgreSQL/Vault connection, migration acceptance, timeout timing, TLS certificate/hostname/handshake acceptance, load/p95 evidence or independent exact-head approval until those are produced on this exact lineage.
Next dependency
Repair the Integration frozen-lock importer atomically on this lineage. Then prove real PostgreSQL + Vault migrations, readiness/startup/shutdown, connection limits, statement/query timeout behavior, idle-error handling, hostile-result cleanup and verified TLS certificate/hostname/handshake behavior on protected lineage. Only after that advance #130 to separately reviewed connect-time DNS/IP-rebinding-safe outbound HTTPS, redirect/proxy policy, signing/idempotency, durable attempt/outcome/retry/dead-letter/recovery, revocation fencing, operator recovery and buyer-visible state.
Keep Draft. No source copy, cross-service SQL, mutable dependency, self-approval, bypass, gate weakening, destructive rebase or force-push.
Refs #130, #205, #235, #241, #242, #243, #244.