Skip to content

fix: restore animation-server, geolocation-server source and globe tours skill#10

Open
jdehorty wants to merge 2 commits intomainfrom
fix/restore-missing-servers-and-skill
Open

fix: restore animation-server, geolocation-server source and globe tours skill#10
jdehorty wants to merge 2 commits intomainfrom
fix/restore-missing-servers-and-skill

Conversation

@jdehorty
Copy link
Contributor

Summary

  • Restored animation-server source from commit f4f1808 with full modular structure (schemas/, tools/, utils/) — includes animation, clock control, and globe lighting tools on port 3006
  • Restored and refactored geolocation-server from commit df0b8fc — split monolithic files into modular pattern matching entity-server, with place search, nearby search, routing, and user location tools on port 3005
  • Restored creating-globe-tours skill with updated tool references (clock_configureclock_control), expanded entity types, and corrected multi-server coordination table
  • Extended shared ResponseEmoji enum with 6 new animation-related values needed by animation-server
  • Updated workspace configuration — added build/dev scripts for animation and geolocation servers, updated root README with new server documentation

Context

PR #7 overwrote PR #6's merge commit, leaving animation-server and geolocation-server with only build/ artifacts (no source) and the globe tours skill completely missing. The clock-server (which only had build artifacts) was removed since its tools were absorbed into animation-server.

Test plan

  • All packages build successfully (pnpm run build from mcp/)
  • Lint passes (pnpm run lint from mcp/)
  • animation-server has full src/ with modular structure matching entity-server pattern
  • geolocation-server has full src/ with modular structure matching entity-server pattern
  • skills/creating-globe-tours/SKILL.md exists with correct tool references
  • No broken cross-package imports

…urs skill

PR #7 overwrote PR #6's merge commit, leaving animation-server and
geolocation-server with only build artifacts (no source) and deleting
the creating-globe-tours skill entirely.

- Recover animation-server source from commit f4f1808 (already
  refactored into modular pattern with split schemas/tools/utils)
- Recover geolocation-server source from commit df0b8fc and refactor
  into modular pattern matching entity-server conventions
- Restore creating-globe-tours SKILL.md with updated tool references
  (clock_configure -> clock_control) and new entity types
- Add animation-specific ResponseEmoji values to shared package
- Update workspace scripts and README for new servers
- Remove dead clock-server (tools absorbed into animation-server)
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