Skip to content

fix: put each tool's title in annotations.title too - #16

Merged
STRML merged 2 commits into
mainfrom
fix/annotation-title
Oct 1, 2026
Merged

STRML merged 2 commits into
mainfrom
fix/annotation-title

Conversation

@STRML

@STRML STRML commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Every tool now carries its title in annotations.title as well as the top-level title. The Claude directory submission portal flagged "Missing title annotation": it reads the older annotation field, and the earlier fix (PR 16 in the private repo) added only the top-level one.

 before                              after
 ──────────────────────────          ───────────────────────────────────────────
 title: "Compare two laps"           title: "Compare two laps"
 annotations: {                      annotations: {
   readOnlyHint, destructiveHint,      readOnlyHint, destructiveHint,
   openWorldHint }                     openWorldHint,
                                       title: "Compare two laps" }

One helper per server (titled(title)) feeds both fields from a single string, so they cannot drift. All seven tools: diagnose_symptom, maintenance_schedule, lookup_procedure, analyze_session, compare_laps, find_time_loss, compare_sessions.

Status

Check Result
listing rows red first (maintenance 12, laps 20) annotations.title equals title failed
maintenance suite 23 of 23
laps suite 65 of 65
package and listing tests 15 of 15
typecheck clean

Test plan

  • both listing rows assert annotations.title === title
  • after deploy, tools/list on both live hosts shows annotations.title, then re-sync the tools in the directory portal

The Claude directory submission portal reported Missing title annotation: it reads annotations.title, while the earlier fix added only the top-level title.

- both servers: a titled(title) helper feeds the top-level title and annotations.title from one string, for all seven tools
- the listing rows (maintenance 12, laps 20) assert annotations.title equals title, red first
- maintenance 23 of 23, laps 65 of 65, package and listing tests 15 of 15
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Next included review available in 1 second.

Check out review usage here.

View limit details

Limit details: You’ve used all 3 included reviews currently available. Your 75 included PR review attempts over the past 7 days set your current allowance at 3 reviews per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 54245961-547f-4cb9-a607-5d70cd6025fa

📥 Commits

Reviewing files that changed from the base of the PR and between 9a2d8a6 and eeac06e.

📒 Files selected for processing (4)
  • servers/laps/e2e/laps.e2e.test.mjs
  • servers/laps/src/tools.ts
  • servers/maintenance/e2e/maintenance.e2e.test.mjs
  • servers/maintenance/src/tools.ts
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@STRML
STRML merged commit 8118878 into main Oct 1, 2026
5 checks passed
@STRML
STRML deleted the fix/annotation-title branch October 1, 2026 18:29
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.

1 participant