Fix Docker build and add zod dependency#196
Merged
Merged
Conversation
- Update Dockerfile to build TypeScript server instead of static site - Install all dependencies (including TypeScript) in builder stage - Copy dist/, server/public/, static/, and registry/ to production image - Remove nginx - now using Node.js Express server directly - Remove unused nginx.conf - Set PORT=8080 for Fly.io compatibility Fixes build error: "tsc: not found" 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The @adcp/client package requires zod but doesn't declare it properly, causing Docker production builds to fail. Added zod as a direct dependency to ensure it's available in production installs. Fixes: "Cannot find module 'zod'" error in Docker container 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove "Built with Docusaurus" from footer - Remove DocuSaurus generator meta tag - Update logo references to use new adcp_logo.svg - Add new AdCP logo to public directory 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Enhanced meta tags and structured data for better discoverability by both traditional search engines and AI language models: - Remove DocuSaurus error banner script (unused bloat) - Enhanced JSON-LD structured data with: - Updated logo reference to adcp_logo.svg - Software version and release notes - Programming language metadata - License information - SearchAction for docs site - FAQ schema with common questions - Founding member organizations - Additional meta tags for AI crawlers: - Author and publisher metadata - Abstract and topic classification - Enhanced robots directives - Twitter card metadata - Coverage and distribution info - Expanded keywords for "agentic advertising" and A2A protocol These changes improve: 1. Traditional SEO for search engines 2. GEO for AI models (Claude, ChatGPT, Perplexity, etc.) 3. Social media preview cards 4. Knowledge graph representation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test Plan
🤖 Generated with Claude Code