Skip to content

spec(intent-4): a registration omitting a required identity field is malformed - #261

Draft
openvoiceos-bot wants to merge 1 commit into
devfrom
spec/intent-4-omitted-identity-field
Draft

spec(intent-4): a registration omitting a required identity field is malformed#261
openvoiceos-bot wants to merge 1 commit into
devfrom
spec/intent-4-omitted-identity-field

Conversation

@openvoiceos-bot

Copy link
Copy Markdown

🤖 Auto-generated by Claude Fable 5.1 (claude-fable-5-1) via Claude Code — NOT human-reviewed. Verify before acting.

Decision for Miro: this adds a MUST to INTENT-4 §3.2 and merges only on his yes.

INTENT-4 marks skill_id, intent_name/entity_name and lang required in the §3.2 table but says nothing about what a consumer does when one is absent; the malformed set in §5.3, §6.3, §7.2 and §12 covers constraints, samples and reserved names only. The new paragraph closes that: a message of §§5–8 omitting a required identity field is malformed, a consumer MUST NOT index or act on it, MUST NOT derive the value from context, topic or another field, and MUST log the rejection at WARN naming the missing field. CHANGELOG entry under INTENT-4.

What it codifies, measured by the harness lane on a real bus: adapt, padatious and m2v already reject an unattributed registration and WARN. padacioso accepts it and derives skill_id from the colon prefix of intent_name, silently; that is the behaviour this clause forbids, and the fourth implementation is the one the clause exists for.

The existing sentence that a message is complete without context.skill_id is unchanged: the source may be absent, the target may not.

Verified against source by model re-check, not human review: engine logs in knowledge/wiki/audits/gate-ledger (harness cells for adapt#76, padatious#153, markov#16); the reading with the silence established by search is knowledge/wiki/audits/architecture/readings-2026-09-09.md §2.

…malformed and is never repaired

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 9, 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.

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