Skip to content

Regenerate deleted recommended skills - #3693

Open
louis4li wants to merge 1 commit into
feature/integratefrom
fix/2026-10-01_channel-nyxid-credential-routing
Open

louis4li wants to merge 1 commit into
feature/integratefrom
fix/2026-10-01_channel-nyxid-credential-routing

Conversation

@louis4li

@louis4li louis4li commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Issue

Closes #3691

Scope

  • Regenerate an Ornn recommended skill only when exact loading returns a typed NotFound.
  • Replace the stale request-local recommended ref and rebuild the request-local catalog.
  • Return generated skill markdown immediately from the creator result, including when durable ref persistence is unavailable.
  • Preserve AccessDenied, digest mismatch, transport, and other exact-load failures.

Verification

  • dotnet test test/Aevatar.GAgents.ChannelRuntime.Tests/Aevatar.GAgents.ChannelRuntime.Tests.csproj --nologo --filter FullyQualifiedName~ChannelNyxIdConnectedServiceInventoryToolSourceTests (52 passed)
  • bash tools/ci/test_stability_guards.sh
  • git diff --check
  • bash tools/ci/architecture_guards.sh (blocked by existing ai_messages.proto conformance revision mismatch)

Notes

  • The real local relay previously reproduced USE_SKILL_NOT_FOUND for the deleted Ornn ref. This change targets that exact failure boundary; no direct fake relay callback was used.

Generated with Claude Code.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@louis4li
louis4li requested a review from jason-aelf as a code owner October 1, 2026 13:04
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

This branch has not been deployed

No deployments
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.

1 participant