Skip to content

Conversation

@AlexanderNZ
Copy link
Contributor

@AlexanderNZ AlexanderNZ commented Jan 29, 2026

This is the bigger cleanup pass: I split the server into focused tool modules and made the tests easier to run for real orgs.

What I changed

  • replaced the monolithic entrypoint with a tool registry and src/index.ts
  • removed teams support from the API surface
  • added a hardened IcePanel client (timeouts, retries, clearer errors)
  • added integration tests and CLI parsing tests
  • removed hardcoded test values and documented required env vars
  • ignored .pnpm-store/ to avoid accidental staging

Why I did it

  • the old file was hard to maintain and extend
  • tests needed a real, repeatable setup instead of baked-in defaults
  • teams support was unused and added noise to the API surface

Test plan

  • not run

Recommended merge order

Eliminate team-related tools and client code, simplify model object handling, and update integration tests to auto-detect write scope with clearer docs on test configuration.
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