Stale "Agentspan" product-name references in agents READMEs 🐛
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 82/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- typescript
- Domain
- documentation
Research direction
Start with the listed README files under examples/agents and run grep -rin agentspan to find the additional comments and documentation references. Replace only the prose and product-name usages, preserving the _agentspan metadata property noted in the issue. Review the diff and confirm the stale user-facing references are gone without changing the serializer property.
Written by the indexing model from the issue text.
Description
The old product name "Agentspan" still appears in several of the agents example READMEs. It should be scrubbed from user-facing docs (the public API/error hierarchy already dropped it — see AgentspanError → ConductorAgentError in the CHANGELOG).
Where
| File | Lines | Snippet |
|---|---|---|
examples/agents/langgraph/README.md |
3, 63, 85, 109, 147, 162, 186 | "Add agentspan metadata", "add agentspan import", "After (Agentspan)" |
examples/agents/vercel-ai/README.md |
68, 113 | "builds a Agentspan Agent", "// agentspan Agent + Runtime" |
examples/agents/adk/README.md |
57 | "// ^^^ add agentspan import" |
examples/agents/openai/README.md |
51 | "// ^^^ add agentspan import" |
Note for whoever fixes this
Not every hit is a product-name leak. The (graph as any)._agentspan = { ... } metadata property (langgraph README, docs/agents/frameworks/langgraph.md, langchain.md) is a real internal property the serializer reads — leave that one alone. Only the prose/product-name usages ("Agentspan", "add agentspan import", "After (Agentspan)") should be reworded.
There are also a few non-README hits (examples/agents/*.ts comments, AGENTS.md, eslint.config.mjs) worth sweeping while you're in there — grep -rin agentspan finds them all.
Refiled on behalf of @maria-shimkovska from conductor-oss/conductor#1536 (moved here since the references live in this repo).
- Dominant language
- TypeScript
- Stars
- 58
- Forks
- 20
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 7
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 conductor-oss/javascript-sdk
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 88/100
conductor-oss/javascript-sdk#140 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
conductor-oss/javascript-sdk#139 ·
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 91/100
conductor-oss/javascript-sdk#138 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
conductor-oss/javascript-sdk#137 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
conductor-oss/javascript-sdk#136 ·
All issues in conductor-oss/javascript-sdk
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 1/5 Under an hour Newbie friendliness 78/100
fullcalendar/fullcalendar#8106 ·