Skip to content

issue-1696: Align permission_result_id "Other" description with standard wording - #2

Merged
Levaj2000 merged 1 commit into
issue-1696-stop-reasonfrom
claude/zen-einstein-lutmec
Oct 13, 2026
Merged

Levaj2000 merged 1 commit into
issue-1696-stop-reasonfrom
claude/zen-einstein-lutmec

Conversation

@Levaj2000

Copy link
Copy Markdown
Owner

Addresses @pagbabian-splunk's review nit on ocsf#1704.

Related Issue:

ocsf#1696

Description of changes:

Updates the Other (99) description for permission_result_id to the standard pattern, so it matches stop_reason_id:

The permission result is not mapped. See the permission_result attribute, which contains a data source specific value.

Changed in both places that carried the old string:

  • objects/ai_status.json: the object-level enum
  • dictionary.json: the default 0/99 enum on permission_result_id

This only changes description text. No CHANGELOG update is needed.

Merging this into issue-1696-stop-reason updates the upstream PR.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RqaVeXeocZxwEmz6VfNJdr


Generated by Claude Code

@Levaj2000
Levaj2000 force-pushed the claude/zen-einstein-lutmec branch from f41719c to 12a9bcc Compare October 13, 2026 00:54
…standard wording

Use the standard "is not mapped. See the <code>permission_result</code>
attribute, which contains a data source specific value." pattern for the
Other (99) value, in both objects/ai_status.json and the dictionary default
enum, matching stop_reason_id. Addresses review nit on ocsf#1704.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RqaVeXeocZxwEmz6VfNJdr
Signed-off-by: Claude <noreply@anthropic.com>
Signed-off-by: Jeff Leva <120221487+Levaj2000@users.noreply.github.com>
@Levaj2000
Levaj2000 force-pushed the claude/zen-einstein-lutmec branch from 12a9bcc to 8c87013 Compare October 13, 2026 00:56
@Levaj2000
Levaj2000 marked this pull request as ready for review October 13, 2026 00:56
@Levaj2000
Levaj2000 merged commit 637fa60 into issue-1696-stop-reason Oct 13, 2026
2 checks passed
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