Skip to content

feat(con): add hidden submitter contribution and complete awards - #473

Merged
abiorh-claw merged 13 commits into
mainfrom
codex/con07-submitter-participant
Oct 5, 2026
Merged

abiorh-claw merged 13 commits into
mainfrom
codex/con07-submitter-participant

Conversation

@Abiorh001

@Abiorh001 Abiorh001 commented Oct 4, 2026 •

Copy link
Copy Markdown
Collaborator

Change

Add the hidden submitter participant needed by shared acceptance. It creates or exactly replays one FinalAcceptance-sourced contribution and the complete frozen set of zero, one or two compensation awards in the caller's root transaction. A deferred PostgreSQL guard rejects missing awards and refuses incomplete retained data during upgrade.

Change record

Boundaries

  • The existing lifecycle fence is supplied through a consumer-owned interface, preserving the future REV → CON dependency. CON uses COMPENSATION's public award port; neither participant commits.
  • Frozen published/retired rules remain authoritative. Replay preserves IDs and rejects incomplete sets without backfill. Compensated replay checks retained correlation; unpaid records have no award correlation to compare.
  • This adds no production registration, acceptance/source writer, TASK transition, AUTH allow, live routing, reviewer participant or payment delivery. Source-authority hardening and shared acceptance composition remain required before activation.
  • The affected current roadmap/specifications and initiative navigation advance to shared acceptance/TASK effects while preserving the independent human-review routing branch and later reviewer work.

Verification

  • Exact candidate 0737624c, reconciled with main f169c9eb; merged MCP administrative reads and CLI profile editing are preserved.
  • Full hosted CI: 8,549 unique tests collected and completed exactly once, zero skips/deselections. The synthetic merge tree matches the candidate; all evidence hashes and database/MinIO cleanup checks pass. MCP, CLI contract and Agent Gates pass.
  • Focused PostgreSQL proof covers all four economics shapes, exact replay, foreign-source rejection, incomplete-award replay, independent-session concurrency, root/savepoint fencing, atomic rollback, deferred SQL completeness and retained-data migration preservation/refusal. Guard-removal probes detect the intended replay and SQL defects. 274 focused pure contract/ownership/lane-evidence checks also passed.
  • Architecture/reuse, security, QA/test-delta, CI-integrity and documentation/product-operations reviews passed on the final candidate; no open internal finding. Ruff, module boundaries, Markdown links, stale wording and Commitrail checks passed.
  • Existing contribution-storage modules and new participant tests use project partitions; all tests, nine lanes, caps and completeness gates are preserved. Slowest lane: 1,148.472 seconds against the unchanged 1,200-second cap. Total backend wall time remains about 22m44s, above the diagnostic timing target.

Remaining boundaries

Future production composition must prove that the canonical fence, CON participant and award owner share the caller's transaction, with genuine source authority and complete TASK/acceptance effects. This hidden participant does not establish those live guarantees. CodeRabbit is rate-limited on this exact head; its green status is not a substantive review. No GitHub review threads are unresolved.

Human approval and merge remain required.

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 32 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4a9ff0e3-637d-493d-acef-c389d2b31ab4
📥 Commits

Reviewing files that changed from the base of the PR and between f169c9e and 0737624.

📒 Files selected for processing (45)
  • .ci/behavior-ownership/partition.v1.json
  • .commitrail/INDEX.md
  • .commitrail/initiatives/WS-ARCH-001/OVERVIEW.md
  • .commitrail/initiatives/WS-ARCH-001/planning/CHUNK_MAP.md
  • .commitrail/initiatives/WS-ARCH-001/planning/PLAN.md
  • .commitrail/initiatives/WS-AUTH-001/OVERVIEW.md
  • .commitrail/initiatives/WS-AUTH-001/planning/CHUNK_MAP.md
  • .commitrail/initiatives/WS-AUTH-001/planning/PLAN.md
  • .commitrail/initiatives/WS-CON-001/OVERVIEW.md
  • .commitrail/initiatives/WS-CON-001/WS-CON-001-07.md
  • .commitrail/initiatives/WS-POL-003/OVERVIEW.md
  • .commitrail/initiatives/WS-POL-003/planning/CHUNK_MAP.md
  • .commitrail/initiatives/WS-POL-003/planning/PLAN.md
  • .commitrail/initiatives/WS-REV-001/OVERVIEW.md
  • README.md
  • backend/alembic/env.py
  • backend/alembic/versions/0019_submitter_award_completeness.py
  • backend/app/modules/compensation/api/__init__.py
  • backend/app/modules/compensation/api/awards.py
  • backend/app/modules/compensation/awards/participant.py
  • backend/app/modules/contributions/api/__init__.py
  • backend/app/modules/contributions/api/participation.py
  • backend/app/modules/contributions/records/participant.py
  • backend/app/modules/contributions/records/repository.py
  • backend/scripts/behavior_ownership.py
  • backend/scripts/test_lane_catalogue.py
  • backend/tests/conftest.py
  • backend/tests/contributions/participation/__init__.py
  • backend/tests/contributions/participation/support.py
  • backend/tests/contributions/participation/test_contracts.py
  • backend/tests/contributions/participation/test_postgresql.py
  • backend/tests/contributions/participation/test_transactions.py
  • backend/tests/contributions/records/support.py
  • backend/tests/contributions/records/test_migration.py
  • backend/tests/contributions/records/test_storage.py
  • backend/tests/test_alembic.py
  • backend/tests/test_behavior_ownership.py
  • backend/tests/test_ci_lane_catalogue.py
  • docs/architecture_data_model.md
  • docs/glossary.md
  • docs/roadmap_status.md
  • docs/spec_artifact_storage_service.md
  • docs/spec_authorization_service.md
  • docs/spec_contribution_compensation.md
  • docs/spec_review_lifecycle.md
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@Abiorh001
Abiorh001 marked this pull request as ready for review October 4, 2026 17:28
@abiorh-claw
abiorh-claw self-requested a review October 5, 2026 02:29
@abiorh-claw
abiorh-claw merged commit 0ba68ea into main Oct 5, 2026
17 checks passed
@abiorh-claw
abiorh-claw deleted the codex/con07-submitter-participant branch October 5, 2026 02:29
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.

2 participants