Skip to content

Fix Glama MCP build and tool introspection - #36

Closed
AjnasNB wants to merge 2 commits into
mainfrom
agent/glama-mcp-release
Closed

AjnasNB wants to merge 2 commits into
mainfrom
agent/glama-mcp-release

Conversation

@AjnasNB

@AjnasNB AjnasNB commented Aug 9, 2026

Copy link
Copy Markdown
Owner

What changed

  • add root glama.json maintainer metadata for repository ownership verification
  • add a digest-pinned Dockerfile.glama that starts the native stdio MCP process instead of the HTTP daemon
  • add npm run check:glama, which performs a real MCP initialize and tools/list exchange and requires all eight public tools
  • exercise both the local process and the built Glama container in CI
  • document the separate Glama build path in the generated MCP manual and discovery surfaces

Why

Glama was building the repository's root daemon image. That image correctly starts the authenticated HTTP browser daemon, but Glama expects a stdio MCP process for protocol introspection, so the listing reported zero tools and could not produce a Glama release.

Impact

The normal daemon Docker image and runtime behavior are unchanged. Glama maintainers can select Dockerfile.glama with the repository root as build context, deploy it, and inspect the MCP schema without a secret. Daemon-backed tool calls still require the documented URL and token.

Validation

  • local npm run check passed
  • 65 tests passed, 1 browser smoke test skipped locally by its existing environment guard
  • GitHub CI built Dockerfile.glama, initialized MCP inside the container, and discovered all 8 registered tools
  • Node 22, 24, and 26; Chromium smoke; packed consumer; hardened daemon container; dependency review; and CodeQL all passed
  • package, site, audit, and tarball gates passed with 0 runtime vulnerabilities

@AjnasNB
AjnasNB marked this pull request as ready for review August 9, 2026 04:04
@AjnasNB
AjnasNB requested a review from cognifyrdotco August 9, 2026 04:04
@AjnasNB
AjnasNB enabled auto-merge (squash) August 9, 2026 04:04
@AjnasNB

AjnasNB commented Aug 9, 2026

Copy link
Copy Markdown
Owner Author

Branch-name migration only: replacement #42 points to exact head abadf562746f9ae52e9adbce0d679486abb9510b on neutral ref fix/glama-mcp-release-20260809. The source tree, commits, base, title/body, review request, and protection requirements are unchanged. Closing this duplicate before deleting only the legacy head ref; no source or main history is rewritten.

@AjnasNB AjnasNB closed this Aug 9, 2026
auto-merge was automatically disabled August 9, 2026 17:30

Pull request was closed

@AjnasNB
AjnasNB deleted the agent/glama-mcp-release branch August 9, 2026 17:30
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