Skip to content

test: fix ordinary app trigger proof inputs - #275

Merged
Joncallim merged 13 commits into
codex/pr198-root-reconciler-wrong-actor-prooffrom
codex/pr198-ordinary-app-trigger-inputs
Jul 27, 2026
Merged

test: fix ordinary app trigger proof inputs#275
Joncallim merged 13 commits into
codex/pr198-root-reconciler-wrong-actor-prooffrom
codex/pr198-ordinary-app-trigger-inputs

Conversation

@Joncallim

Copy link
Copy Markdown
Owner

Scope

Repairs the ordinary application trigger/default proof after the six PUBLIC routine revokes. The final assertion now uses psql stdin with typed UUID variables, rather than a command-mode string where psql cannot expand :'project_id' / :'package_id'.

The proof still authenticates as forge_app_test and verifies project and package pointer creation plus eight projection heads. No production behavior or ACL changes.

Validation

  • bash -n scripts/ci/prove-migration-0027-ordinary-app-trigger-writes.sh
  • npm test -- --run __tests__/project-root-reconciliation.test.ts (20 passed)
  • npm test (1,629 passed, 19 skipped)
  • npx eslint --max-warnings=0 __tests__/project-root-reconciliation.test.ts
  • npx tsc --noEmit
  • git diff --check

The PostgreSQL role proof remains mandatory Web CI coverage; role URLs are not locally provisioned.

@Joncallim
Joncallim marked this pull request as ready for review July 27, 2026 21:07
@Joncallim
Joncallim merged commit e422b46 into codex/pr198-root-reconciler-wrong-actor-proof Jul 27, 2026
2 of 3 checks passed
@Joncallim
Joncallim deleted the codex/pr198-ordinary-app-trigger-inputs branch August 2, 2026 18:22
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