Skip to content

Remove HTTP transport, simplify to stdio-only#1

Merged
andreeapopvisma merged 2 commits intomasterfrom
remove-http-transport
Feb 23, 2026
Merged

Remove HTTP transport, simplify to stdio-only#1
andreeapopvisma merged 2 commits intomasterfrom
remove-http-transport

Conversation

@andreeapopvisma
Copy link
Contributor

Summary

  • Remove HTTP/SSE transport from server — stdio only
  • Remove HTTP-related env vars, EXPOSE, and HEALTHCHECK from Dockerfile
  • Remove ECS deploy job from CI pipeline (keep build + push to ECR)
  • Add GITHUB_TOKEN env var for cloning the private Development-Documentation repo
  • Update README to reflect changes

Test plan

  • npm run build compiles without errors
  • npx tsx src/index.ts starts in stdio mode
  • Docker build succeeds
  • CI pipeline builds and pushes image to ECR

🤖 Generated with Claude Code

andreeapopvisma and others added 2 commits February 23, 2026 12:18
- Remove HTTP/SSE transport from index.ts (stdio only)
- Remove HTTP-related env vars, EXPOSE, and HEALTHCHECK from Dockerfile
- Remove ECS deploy job from CI pipeline (keep build+push to ECR)
- Add GITHUB_TOKEN env var for cloning private docs repo
- Update README to reflect stdio-only usage

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…bc support

Alpine uses musl which is incompatible with onnxruntime-node native binaries.
Model cache is already included via node_modules, so remove the separate cache
copy step.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@andreeapopvisma andreeapopvisma merged commit a169a24 into master Feb 23, 2026
2 checks passed
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