Skip to content

Enhancement: Simplify twdesk and twspaces tool descriptions#303

Merged
tw-garymeehan merged 1 commit into
mainfrom
gary/twdesk-strip-marketing
May 6, 2026
Merged

Enhancement: Simplify twdesk and twspaces tool descriptions#303
tw-garymeehan merged 1 commit into
mainfrom
gary/twdesk-strip-marketing

Conversation

@tw-garymeehan
Copy link
Copy Markdown
Contributor

Summary

  • Strip marketing prose (Useful for…, Enables users to audit/analyze/synchronize…, adapting Desk to evolving support processes) from every twdesk tool description; align twspaces tone with the same terse template.
  • Add light-touch product disambiguators (Desk tag, Spaces tag, page comment, Desk company) on the few tools that share a bare suffix with twprojects, so the LLM has a non-prefix signal to pick by.
  • Delete twdesk/desk.go and twspaces/spaces_desc.go (now-unused marketing blurbs); replace toolset references with terse constants matching the existing peopleDescription / timeDescription style.

Token savings (mcp-tokens vs main)

base (main)        33,925
current            32,413
delta              -1,512  (-4.46%)

twdesk + twspaces tools alone: 11,937 → 10,405 (−12.8%).

Test plan

  • gofmt -s -w ., go vet ./..., go build ./... clean
  • go test ./internal/twdesk/... ./internal/twspaces/... passes
  • go run ./cmd/mcp-tokens -base main confirms savings
  • Sanity-check tool descriptions render correctly in MCP inspector / consumer

🤖 Generated with Claude Code

Strip marketing prose ("Useful for...", "Enables users to audit/analyze/
synchronize...", "adapting Desk to evolving support processes") from every
twdesk tool description and align twspaces tone with the same terse template.
Add light-touch product disambiguators ("Desk tag", "Spaces tag", "page
comment", "Desk company") on the few tools that share a bare suffix with
twprojects so the LLM has a non-prefix signal to pick by.

Delete twdesk/desk.go and twspaces/spaces_desc.go (now-unused marketing
blurbs) and replace their toolset references with terse constants matching
the existing peopleDescription / timeDescription style.

Token savings vs main (mcp-tokens): 33,925 -> 32,413 (-1,512, -4.5%).
twdesk + twspaces tools alone: 11,937 -> 10,405 (-12.8%).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tw-garymeehan tw-garymeehan requested a review from a team as a code owner May 6, 2026 10:59
@tw-garymeehan tw-garymeehan merged commit 4bd4fc6 into main May 6, 2026
3 checks passed
@tw-garymeehan tw-garymeehan deleted the gary/twdesk-strip-marketing branch May 6, 2026 12:51
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.

2 participants