Pathfinder: remove the unused exploreDocs helper
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 86/100
- Issue type
- Refactor
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- typescript
Research direction
Start in packages/outpost/ai/src/pathfinder.ts and confirm exploreDocs and queryKnowledgeBase have no production callers beyond the references described. Update packages/outpost/ai/src/pipeline.test.ts to remove their vi.fn() entries, then run the pipeline tests. Done means the unused methods and mocks are gone and the existing tests pass.
Written by the indexing model from the issue text.
Description
Spun out of the #112 review (non-blocking cleanup).
PathfinderClient.exploreDocs and queryKnowledgeBase have no production callers — pipeline.ts only uses searchDocs and disconnect; the two methods appear solely as vi.fn() entries in pipeline.test.ts. #112 also removed their direct unit tests (correctly — the shared callTool/parseSearchResults path is covered via searchDocs). Rather than carry untested, unused API surface, remove both methods (and their mock entries), or document why they're being kept if there's a planned consumer.
Files: packages/outpost/ai/src/pathfinder.ts, packages/outpost/ai/src/pipeline.test.ts.
- Dominant language
- TypeScript
- Stars
- 7
- Forks
- 3
- Avg merge
- 7d 16h
- Merged PRs (30d)
- 15
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from CopilotKit/outpost
-
area: docs area: security roadmap: now
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
CopilotKit/outpost#277 ·
-
area: infrastructure roadmap roadmap: later
Difficulty 1/5 Under an hour Newbie friendliness 74/100
CopilotKit/outpost#179 ·
-
area: ai roadmap roadmap: now
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
CopilotKit/outpost#145 ·
-
area: integrations priority: low roadmap roadmap: later
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
CopilotKit/outpost#123 · 2 comments ·
-
area: infrastructure bug priority: low roadmap roadmap: later
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
CopilotKit/outpost#120 · 2 comments ·
All issues in CopilotKit/outpost
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
calcite-components needs triage refactor
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Esri/calcite-design-system#15203 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
danielmiessler/LifeOS#2218 ·