refactor(go-agent): unify tool-backed component creation

[Hz-186]

Summary

• Add a shared factory for tool-backed components
• Refactor SearXNG and GitHub to use declarative registration
• Keep existing invocation and output behavior unchanged

Tested:
bash build.sh --test ./internal/agent/component/...

[coderabbitai]

📝 Walkthrough

Walkthrough

The component package now provides shared tool-to-component factory wiring. GitHub and SearXNG adopt this wiring, tool builders accept additional node parameters, and fixture initialization updates legacy, BGPT, and GitHub registrations.

Changes

Tool component wiring

Layer / File(s)	Summary
Shared tool factory and registration internal/agent/component/tool2component.go	Defines shared tool invoker/spec types, builds tools, validates invokers, wraps components, and registers factories.
GitHub and SearXNG adapter migration internal/agent/component/github.go, internal/agent/component/searxng.go, internal/agent/tool/registry.go, internal/agent/.../*_test.go	Migrates both components to shared wiring, registers tool specifications, and allows extra Python-node-style parameters while retaining supported-parameter validation.
Fixture registry alignment internal/agent/component/fixture_stubs.go	Adds Iteration, IterationItem, and BGPT registrations and removes the fixture GitHub registration.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Init as Component init
  participant Registry as Component registry
  participant Factory as Tool component factory
  participant Tool as agenttool.BuildByName
  participant Component as GitHub or SearXNG component
  Init->>Registry: Register tool component specification
  Registry->>Factory: Create component factory
  Factory->>Tool: Build tool with params
  Tool-->>Factory: Return toolInvoker
  Factory->>Component: Wrap invoker
  Component-->>Factory: Return Component

Loading

Possibly related PRs

• infiniflow/ragflow#16784: Modifies the same fixture component registration list.
• infiniflow/ragflow#16849: Also changes GitHub component wiring and registration.
• infiniflow/ragflow#16854: Overlaps with the SearXNG tool and component contract changes.

Suggested labels: go

Poem

A rabbit hops through tools anew,
With GitHub tags and SearXNG too.
Iterations join the registry line,
BGPT sparkles, components align.
Shared invokers softly run—
Wiring work is neatly done! 🐇

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
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.
Title check	✅ Passed	The title is concise and accurately summarizes the main change: unifying tool-backed component creation.
Description check	✅ Passed	The description includes a clear Summary and testing note, which satisfies the template with enough context for review.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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