Skip to content

Remove unreferenced $defs entries from main workflow schema - #55699

Merged
pelikhan merged 2 commits into
mainfrom
copilot/schema-audit-fix-32812348684
Aug 25, 2026
Merged

pelikhan merged 2 commits into
mainfrom
copilot/schema-audit-fix-32812348684

Conversation

Copilot AI commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

The daily schema consistency audit flagged an unreferenced $defs entry in pkg/parser/schemas/main_workflow_schema.json.

Root cause

max_effective_tokens_limit and positive_effective_tokens_limit were leftover definitions from before the "Effective Tokens" cost metric was renamed to "AI Credits" (see ADR-36496). The max-effective-tokens frontmatter field was renamed to max-ai-credits, which was migrated to reference the newer max_ai_credits_limit / positive_ai_credits_limit defs — but the old defs were never cleaned up, leaving them dangling with zero references in the schema.

Changes

  • Removed the orphaned max_effective_tokens_limit and positive_effective_tokens_limit $defs entries from main_workflow_schema.json
  • No property or validation behavior changes — these defs were not referenced by any schema property
-    "positive_effective_tokens_limit": {
-      "oneOf": [
-        { "type": "integer", "minimum": 1 },
-        { "$ref": "#/$defs/positive_integer_with_km_suffix_or_expression" }
-      ]
-    },
     "positive_ai_credits_limit": {
       ...
     },
-    "max_effective_tokens_limit": {
-      "description": "Maximum effective-token (ET) budget for AWF API proxy enforcement...",
-      "oneOf": [
-        { "$ref": "#/$defs/positive_effective_tokens_limit" },
-        { "type": "integer", "maximum": -1 }
-      ]
-    },
     "max_daily_ai_credits_limit": {
       ...
     },

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix unreferenced $defs entry in daily schema report Remove unreferenced $defs entries from main workflow schema Aug 25, 2026
Copilot AI requested a review from pelikhan August 25, 2026 06:49
@pelikhan
pelikhan marked this pull request as ready for review August 25, 2026 06:51
Copilot AI balanced review requested due to automatic review settings August 25, 2026 06:51
@pelikhan
pelikhan merged commit e2f9ab5 into main Aug 25, 2026
@pelikhan
pelikhan deleted the copilot/schema-audit-fix-32812348684 branch August 25, 2026 06:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes obsolete Effective Tokens schema definitions after migration to AI Credits.

Changes:

  • Removes both legacy, unused $defs entries.
  • Preserves existing schema validation behavior.
Show a summary per file
File Description
pkg/parser/schemas/main_workflow_schema.json Removes unreferenced Effective Tokens definitions.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions github-actions Bot mentioned this pull request Aug 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.8

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.

[schema-audit] Daily Schema Consistency Report — 32812348684

3 participants