Skip to content

Evaluate using mini-bc-agent - #7

Merged
Sun Haoran (haoranpb) merged 10 commits into
mainfrom
features/mini-bc-agent
Oct 5, 2025
Merged

Evaluate using mini-bc-agent#7
Sun Haoran (haoranpb) merged 10 commits into
mainfrom
features/mini-bc-agent

Conversation

@haoranpb

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread scripts/python/bc_environment.py Fixed
Comment thread scripts/python/bc_environment.py Dismissed
@haoranpb
Sun Haoran (haoranpb) merged commit 327d35f into main Oct 5, 2025
5 checks passed
@haoranpb
Sun Haoran (haoranpb) deleted the features/mini-bc-agent branch October 5, 2025 13:30
Marko Aleksandric (AleksandricMarko) added a commit that referenced this pull request Jul 31, 2026
…ding

Applies Haoran's review feedback for the extensibility-request categories:

- Rename to explicit `extensibility-request-implement` / `extensibility-request-triage`
  across category ids, enum members, classes, dataset files and module files (#1/#2/#4).
- Slim both prompt templates to reflect real-world usage; category-specific rules now
  live in the ai-ext-fix / argus-triage skills (#3).
- Merge duplicate `bcbench.results` import in commands/evaluate.py (#5).
- Uptake #761: both ext entries subclass `RepoGroundedEntry` (#6/#7).
- Triage `patch` defaults to None; stripped from the dataset (#8).
- Constrain triage label fields with a `ManagedLabel` Literal to catch dataset typos (#9).
- Move the ext dataset classes into dataset/extensibility_request.py, like code review (#10).
- Drop `fetch_commit_if_missing` from both ext pipelines to match the other pipelines (#11).
- Keep ext-implement judge-only for now; container build/publish noted as future work (#12).
- Grade triage with the NL2AL LMChecklist instead of the code-review-style hybrid:
  `ExtRequestTriageResult` removed in favour of `JudgeBasedEvaluationResult`, expected_*
  fields replaced by an `expected` checklist, evaluators `lm_checklist`, core score
  `test_passed`; 17 triage entries migrated (#13/#14/#15/#16).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: fb712366-b026-470c-9d09-1bd9e9f52a31
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