Fix registration error when setup

[archlitchi]

Add prefix 'passthrough:///' to avoid connection error

Summary by CodeRabbit

• Bug Fixes
  • Improved local plugin connections over Unix sockets by creating the underlying gRPC client with a passthrough target derived from the socket path.
  • Plugin startup now reports connection-creation failures instead of silently ignoring them.
  • Connection readiness is handled more robustly: the system waits until the connection is ready, fails fast on transient/shutdown states, and returns a clearer timeout error (instead of a generic context timeout).

[hami-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: archlitchi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [archlitchi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[coderabbitai]

📝 Walkthrough

Walkthrough

PluginServer.dial now creates the gRPC client with a passthrough:/// Unix socket target, returns client निर्माण errors, polls connection state until connectivity.Ready, and emits a specific timeout error on deadline.

Changes

gRPC Dial Fix

Layer / File(s)	Summary
gRPC dial target, error handling, and state wait internal/server/register.go	Builds a passthrough:/// target from the Unix socket path, checks grpc.NewClient error, polls connection state until connectivity.Ready, fails fast on TransientFailure or Shutdown, and returns a formatted timeout error.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐇 I hopped through the dial with a passthrough trail,
Ready came clear, no errors to veil.
On timeout I whisper a path-specific song,
The socket connection now knows where it belongs.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title is clearly related to the change: it describes a setup/registration error fix, which matches the registration connection update.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix_register

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@internal/server/register.go`:
- Around line 177-180: The registration flow in NewClient is waiting on
connectivity.Idle before any dialing starts, so it can hang until the context
expires. In the connection setup path that uses grpc.NewClient, call c.Connect()
first, then wait for the channel to reach connectivity.Ready before returning.
Keep the existing timeout/error handling and close the client on failure.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 38ba0b82-dad3-4aff-9073-d853ca0bb838

📥 Commits

Reviewing files that changed from the base of the PR and between ce9ab91 and b54df69.

📒 Files selected for processing (1)

• internal/server/register.go

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@internal/server/register.go`:
- Around line 183-186: The connection wait logic in registerServer currently
treats connectivity.TransientFailure as terminal, but it should keep waiting
until the deadline and only exit early on connectivity.Shutdown. Update the
state-handling branch in the connect/wait loop so TransientFailure is ignored or
retried, while Shutdown still closes the connection and returns an error; use
the existing c.Close, state checks, and deadline-based wait flow to locate the
fix.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 143e6ae1-cc46-41bc-966b-e01f1d1cda70

📥 Commits

Reviewing files that changed from the base of the PR and between b54df69 and 0920f23.

📒 Files selected for processing (1)

• internal/server/register.go