Skip to content

Report skill catalog truncation during rendering - #34785

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e73fdbcf77ac73c7209100ff6c9b5862b6a59a54
Jul 22, 2026
Merged

Report skill catalog truncation during rendering#34785
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e73fdbcf77ac73c7209100ff6c9b5862b6a59a54

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 22, 2026

Copy link
Copy Markdown

Report skill catalog truncation during rendering

What changed

  • Return a SkillRenderReport with total, included, and omitted skill counts,
    plus truncated description counts and character totals.
  • Keep fragment construction separate from rendering so callers can inspect the
    report even when the metadata budget is too small to emit a fragment.

Testing

  • Cover partial description truncation, fully omitted entries, and catalogs
    whose omission marker does not fit the budget.

## What changed

- Return a `SkillRenderReport` with total, included, and omitted skill counts,
  plus truncated description counts and character totals.
- Keep fragment construction separate from rendering so callers can inspect the
  report even when the metadata budget is too small to emit a fragment.

## Testing

- Cover partial description truncation, fully omitted entries, and catalogs
  whose omission marker does not fit the budget.

GitOrigin-RevId: e73fdbcf77ac73c7209100ff6c9b5862b6a59a54
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/e73fdbcf77ac73c7209100ff6c9b5862b6a59a54 branch from 0fd005e to fe6aa9d Compare July 22, 2026 16:27
@copyberry
copyberry Bot merged commit fe6aa9d into main Jul 22, 2026
18 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/e73fdbcf77ac73c7209100ff6c9b5862b6a59a54 branch July 22, 2026 16:28
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant