fix: add data source integrity rule to delegation contract #14

Merged
jerome merged 1 commits from feat/data-source-integrity-rule into master 2026-07-12 18:09:16 +00:00
Owner

What

Adds a hard rule: workers MUST use the data provided in their task context. They MUST NOT fetch their own data from external sources unless explicitly told to.

Why

E2E delegation test revealed the #1 source of cross-worker inconsistency: a writer worker queried the Proxmox API directly instead of formatting the raw SSH data it was given, producing a report with fabricated node IPs, phantom VMs, and false 'online' statuses.

Changes

  • New Data Source Integrity section (hard rule, not recommendation)
  • Dispatch guidance with example showing explicit data-source instruction
  • Anti-pattern entry: workers fetching their own data sources
  • Orphan pre-merge file removed

Test Evidence

E2E test dispatched devops → writer → review. Review worker caught all discrepancies that this rule would prevent.

## What Adds a hard rule: workers MUST use the data provided in their task context. They MUST NOT fetch their own data from external sources unless explicitly told to. ## Why E2E delegation test revealed the #1 source of cross-worker inconsistency: a writer worker queried the Proxmox API directly instead of formatting the raw SSH data it was given, producing a report with fabricated node IPs, phantom VMs, and false 'online' statuses. ## Changes - New **Data Source Integrity** section (hard rule, not recommendation) - Dispatch guidance with example showing explicit data-source instruction - Anti-pattern entry: workers fetching their own data sources - Orphan pre-merge file removed ## Test Evidence E2E test dispatched devops → writer → review. Review worker caught all discrepancies that this rule would prevent.
mumuni-bot added 1 commit 2026-07-11 22:54:47 +00:00
fix: add data source integrity rule to delegation contract
PR Pipeline — Authorize → Validate → Review → Merge / auth (pull_request) Successful in 5s
PR Pipeline — Authorize → Validate → Review → Merge / validate (pull_request) Successful in 2s
PR Pipeline — Authorize → Validate → Review → Merge / lint (pull_request) Successful in 2s
PR Pipeline — Authorize → Validate → Review → Merge / ai-review (pull_request) Successful in 2s
PR Pipeline — Authorize → Validate → Review → Merge / gate (pull_request) Successful in 1s
fdb22948d9
Workers MUST use provided input data, not fetch external sources.
Adds: Data Source Integrity section, dispatch guidance example,
anti-pattern entry. Root cause of e2e test discrepancies where
writer worker queried Proxmox API instead of using raw data.
jerome merged commit 22eaaf4254 into master 2026-07-12 18:09:16 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SyslogSolution/prose-contracts#14