Complete Prime research claim evidence#74
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: da360c8b17
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ) | ||
| if not tool_style: | ||
| return predicted, oracle | ||
| primary_prefix = "kube-linter/" if fixture_type == "k8s" else "semgrep/" |
There was a problem hiding this comment.
Preserve Terraform type before primary-tool filtering
For Hub/build E2 examples, _convert_e2_format() serializes only info into answer, while the dataset schema stores the language under meta.lang; those Terraform answers therefore reach _oracle_from_answer() without fixture_type and default to k8s. When a correct Terraform completion and oracle both use semgrep/... IDs, this line selects the kube-linter/ prefix and filters both sides to empty, so executable and tool-observation rewards become 0 for the Terraform split. Please carry meta.lang/fixture_type into the answer or infer the primary tool from the IDs before filtering.
Useful? React with 👍 / 👎.
Summary
Validation
Notes