Skip to content

docs(intent-4): §7.1 table restates the entity identity fields - #260

Draft
openvoiceos-bot wants to merge 1 commit into
devfrom
docs/intent-4-entity-table-identity
Draft

docs(intent-4): §7.1 table restates the entity identity fields#260
openvoiceos-bot wants to merge 1 commit into
devfrom
docs/intent-4-entity-table-identity

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.

INTENT-4 §7.1's field table restates skill_id and lang as required and says the entity identity is (skill_id, entity_name, lang) with per-language replacement. Both facts already follow from §3.2 ("For an entity, intent_name is replaced by entity_name" and "Replacement is also per-language"); the table now says so where a reader implementing entity registration looks. Editorial; no normative change, no CHANGELOG entry.

Why: two implementations keyed an entity registry on the name alone and let the last language overwrite the others. Both read §7 and neither followed the pointer to §3.2.

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