Skip to content

Add C++ overload conversion ranking#1641

Closed
azizur100389 wants to merge 1 commit into
abhigyanpatwari:mainfrom
azizur100389:codex/cpp-overload-conversion-ranking
Closed

Add C++ overload conversion ranking#1641
azizur100389 wants to merge 1 commit into
abhigyanpatwari:mainfrom
azizur100389:codex/cpp-overload-conversion-ranking

Conversation

@azizur100389

Copy link
Copy Markdown
Contributor

Summary

  • add a C++ primitive overload conversion ranker for exact, promotion, and standard conversion ranks
  • thread the ranker through overload narrowing for C++ resolver call paths
  • add unit coverage and C++ resolver fixtures for int/double overload selection

Fixes #1578

Tests

  • npx vitest run test/unit/scope-resolution/pick-implicit-this-overload.test.ts test/unit/scope-resolution/overload-narrowing.test.ts
  • npx vitest run test/integration/resolvers/cpp.test.ts -t "C\+\+ overload"
  • npx vitest run test/integration/resolvers/cpp.test.ts
  • npx tsc --noEmit
  • git diff --check

Note: full parallel npm test still shows existing CLI e2e stdout/EPIPE failures from shared test state; test/integration/cli-e2e.test.ts passes in isolation, and test/integration/mcp/server-startup.test.ts passes in isolation. A sequential full-suite run exceeded the local 30-minute timeout.

@vercel

vercel Bot commented May 16, 2026

Copy link
Copy Markdown

@azizur100389 is attempting to deploy a commit to the NexusCore Team on Vercel.

A member of the Team first needs to authorize it.

@azizur100389

Copy link
Copy Markdown
Contributor Author

Closing as superseded by upstream PR #1606, which has already merged and closed #1578. This branch now conflicts with the merged implementation, so keeping it open would duplicate the same overload-ranking work.

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.

cpp overload resolution: standard-conversion-sequence ranking

1 participant