Skip to content

Update P/Invoke skill for cross platform declaration#798

Open
huoyaoyuan wants to merge 1 commit into
dotnet:mainfrom
huoyaoyuan:patch-1
Open

Update P/Invoke skill for cross platform declaration#798
huoyaoyuan wants to merge 1 commit into
dotnet:mainfrom
huoyaoyuan:patch-1

Conversation

@huoyaoyuan

@huoyaoyuan huoyaoyuan commented Jun 22, 2026

Copy link
Copy Markdown
Member

Add cases for default naming convention.
Prefer OperatingSystem api over OSPlatform api.

@AaronRobinsonMSFT

@huoyaoyuan huoyaoyuan requested review from a team as code owners June 22, 2026 06:44
@github-actions github-actions Bot added the pr-state/ready-for-eval PR is mergeable and awaiting evaluation label Jun 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Skill Validation Results

Skill Scenario Quality Skills Loaded Overfit Verdict
dotnet-pinvoke Generate LibraryImport declaration from C header (.NET 8+) 5.0/5 → 5.0/5 ✅ dotnet-pinvoke; tools: skill ✅ 0.11
dotnet-pinvoke Generate LibraryImport declaration from C header (.NET Framework) 4.3/5 → 5.0/5 🟢 ✅ dotnet-pinvoke; tools: skill ✅ 0.11 [1]

[1] ⚠️ High run-to-run variance (CV=90%) — consider re-running with --runs 5

Model: claude-opus-4.6 | Judge: claude-opus-4.6

🔍 Full Results - additional metrics and failure investigation steps

▶ Sessions Visualisation -- interactive replay of all evaluation sessions
📊 Session Analytics (preview) -- aggregated metrics across evaluation sessions

@github-actions github-actions Bot added waiting-on-review PR state label and removed pr-state/ready-for-eval PR is mergeable and awaiting evaluation labels Jun 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Evaluation passed for 13fd251. cc @dotnet/appmodel @dotnet/skills-csharp-language-reviewers — please review.

@github-actions github-actions Bot added ready-to-merge PR state label and removed waiting-on-review PR state label labels Jun 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Approved by @AaronRobinsonMSFT. cc @dotnet/skills-merge-approvers — ready to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge PR state label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants