Skip to content

[CI] Reduce Claude review analysis turns - #2020

Open
jayhenry wants to merge 1 commit into
InternLM:mainfrom
jayhenry:claude-review-prompt-efficiency
Open

[CI] Reduce Claude review analysis turns#2020
jayhenry wants to merge 1 commit into
InternLM:mainfrom
jayhenry:claude-review-prompt-efficiency

Conversation

@jayhenry

Copy link
Copy Markdown
Collaborator

Summary

  • Document the prepared JSON/JSONL/TSV bundle schema in the review prompt.
  • Combine manifest, PR metadata, changed files, and diff offsets into one initial jq call.
  • Defer discussion deduplication until Warning+ candidates exist, then use one targeted TSV query and an optional JSONL follow-up.
  • Provide valid git grep forms with all options before -e <pattern> and avoid pipelines that hide Git failures.

Motivation

Run 31704822761 used 30 analysis turns. Three initial calls separately loaded snapshot metadata and file inventory, the initial discussion preview produced 44.7 KB before truncation, and two git grep calls failed because -A / -l appeared after the pattern. This prompt change targets those avoidable turns without lowering the external 34-turn limit or the 28-call analysis budget.

Validation

  • Parsed .github/workflows/claude-general.yml and asserted the prompt contract.
  • Executed the aggregate jq expression against representative manifest, PR, files, and diff-index data.
  • git diff --check upstream/main...HEAD

Document the prepared bundle schema, combine initial inventory into one jq call, defer targeted discussion deduplication, and provide valid git grep command forms.
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.

1 participant