Skip to content

Add translation-bench gold-action validation helper - #2859

Merged
Dominic Nguyen (datduyng) merged 8 commits into
mainfrom
domnguyen/tb-gold-action-validation
Aug 13, 2026
Merged

Add translation-bench gold-action validation helper#2859
Dominic Nguyen (datduyng) merged 8 commits into
mainfrom
domnguyen/tb-gold-action-validation

Conversation

@datduyng

Copy link
Copy Markdown
Contributor

Validate gold actions after restoring required empty parameters and single-literal union fields.

- Restore required empty parameters after placeholder stripping
- Inject required single-literal string-union fields before validateAction
- Export the helper from the synthesizer barrel
@datduyng
Dominic Nguyen (datduyng) marked this pull request as ready for review August 12, 2026 21:40
Comment thread ts/packages/benchmarks/src/translationBench/synthesizer/actionValidation.ts Outdated
Address review feedback on the gold-action validation helper:
- Call the helper from generationCandidate and benchmark validation paths
- Resolve type-reference aliases before injecting single-literal fields
- Add unit coverage next to goldParameterHygiene
@datduyng
Dominic Nguyen (datduyng) added this pull request to the merge queue Aug 12, 2026
@datduyng
Dominic Nguyen (datduyng) removed this pull request from the merge queue due to a manual request Aug 12, 2026
@datduyng
Dominic Nguyen (datduyng) added this pull request to the merge queue Aug 12, 2026
@datduyng
Dominic Nguyen (datduyng) removed this pull request from the merge queue due to a manual request Aug 12, 2026
@datduyng
Dominic Nguyen (datduyng) added this pull request to the merge queue Aug 12, 2026
@datduyng
Dominic Nguyen (datduyng) removed this pull request from the merge queue due to a manual request Aug 12, 2026
@datduyng
Dominic Nguyen (datduyng) added this pull request to the merge queue Aug 13, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 13, 2026
@datduyng
Dominic Nguyen (datduyng) added this pull request to the merge queue Aug 13, 2026
Merged via the queue into main with commit 3ee5b5d Aug 13, 2026
29 of 30 checks passed
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