[AAASM-3415] 🔧 (native): Forward teamId/parentAgentId on native register

[Chisanan232]

Target

• Task summary:

  When the Node SDK moved to native gRPC Register (Epic AAASM-3395 / AAASM-3403), the native register path stopped forwarding the lineage/team fields the old REST register sent. teamId drives team-budget attribution; parentAgentId drives the topology graph. This restores forwarding over the native path.

• Task tickets:

  • Task ID: AAASM-3415.
  • Relative task IDs:
    • go-sdk / aa-ffi-go lineage forwarding — deferred follow-up (go-sdk is being edited concurrently by AAASM-3404).
  • Relative PRs:
    • agent-assembly: [AAASM-3415] 🐛 (aa-sdk-client): Forward team_id/parent_agent_id on native Register agent-assembly#1160 (carries Closes AAASM-3415)
    • python-sdk: [AAASM-3415] 🔧 (native): Forward team_id/parent_agent_id on native register python-sdk#149

• Key point change:

  • native/aa-ffi-node RegisterOptions gains team_id / parent_agent_id; register sets them on the aa-sdk-client AssemblyConfig. index.d.ts regenerated.
  • TS RegisterOptions gains teamId / parentAgentId; buildRegisterOptions sets each only when present so an unset field stays absent.
  • AssemblyConfig (TS) already exposed teamId / parentAgentId; this wires them through to register.
  • Bumped the shared aa-sdk-client / aa-proto git-SHA pin to the agent-assembly AAASM-3415 commit that adds the AssemblyConfig fields. No proto change was needed (the wire already carries both fields).

Effecting Scope

• Action Types:

  • ✏️ Modifying existing something
    • 🟢 No breaking change
  • 🔧 Fixing bug

• Scopes:

  • 🪡 API client and transport
  • 🫀 Data model and types
  • 🧪 Testing
    • 🧪 Unit testing
  • 🚀 Building
    • 🔗 Dependencies

• Additional description:

  pnpm native:build (against the new pin) · pnpm typecheck · pnpm native:check-types · pnpm lint · pnpm test → 307 passed / 2 skipped (incl. 2 new forwarding tests in tests/native-register.test.ts). cargo clippy --all-targets -- -D warnings on the shim clean. The shim's 2-space style is preserved (the diff is a minimal +11 lines — cargo fmt's default 4-space reformat was not applied as the repo has no rustfmt.toml and CI does not gate on it).

Description

• New parameters are optional and additive — None / omitted leaves the agent team-unscoped / root, preserving current behaviour.
• Cross-repo pin dependency: the git-SHA pin points at agent-assembly's AAASM-3415 branch tip (209572bc…); move it to the squash-merge commit once PR #1160 merges. Not hard-blocked since no proto change was needed.

🤖 Generated with Claude Code

[codecov]

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/core/init-assembly.ts	66.66%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[Chisanan232]

Claude Code — PR review (AAASM-3415, node-sdk)

Verdict: ✅ approve — but merge AFTER agent-assembly #1160 + re-pin. CI green (quality, pin-consistency, module-smoke, Analyze); only codecov/patch red (Rust-shim lines uncovered by JS tests — advisory, non-blocking).

• napi RegisterOptions + TS RegisterOptions/buildRegisterOptions forward teamId/parentAgentId; index.d.ts regenerated. pnpm test 307 pass/2 skip (2 new); typecheck/lint/clippy clean. Kept the shim's 2-space style (minimal +11 diff; repo doesn't gate on rustfmt). ✅

⚠️ Pin dependency: pins aa-sdk-client/aa-proto to agent-assembly branch-tip 209572bc. After #1160 squash-merges, re-pin to the merge commit + re-push (pin-consistency gate will re-run), THEN merge. (I can handle the re-pin once #1160 is in.)

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Chisanan232]

Claude Code — re-pin + test pass (AAASM-3415, node)

✅ Re-pin clean + coverage strengthened — merge-ready. Re-pinned aa-sdk-client/aa-proto → merged fdff8e40; aa-* crates share one git rev (pin-consistency) green; all 3 agent-assembly git deps on the single rev. pnpm native:build rebuilt with no index.d.ts drift.

New tests/register-lineage-forwarding.test.ts (7 tests) covers: full config→RegisterOptions exact shape; only-teamId (parent omitted); only-parentAgentId (team omitted); neither; unicode round-trip; long-value no-truncation; empty-string teamId treated as unset — all via the public initAssembly path. Suite 314 pass / 2 skip; typecheck + lint clean. Only codecov/patch red (advisory). Good to merge.