Skip to content

fix(docker): fix Docker build and move Docker to collapsible README section#262

Merged
jlia0 merged 3 commits intomainfrom
jlia0/fix-readme-docker
Mar 26, 2026
Merged

fix(docker): fix Docker build and move Docker to collapsible README section#262
jlia0 merged 3 commits intomainfrom
jlia0/fix-readme-docker

Conversation

@jlia0
Copy link
Copy Markdown
Collaborator

@jlia0 jlia0 commented Mar 25, 2026

Description

Fix broken Docker build and reorganize README so Quick Start focuses on the install script + tinyagi CLI flow, with Docker as a collapsible section.

Changes

  • Skip Puppeteer browser download in builder stage (PUPPETEER_SKIP_DOWNLOAD=true) — system Chromium is installed in the production stage
  • Add missing tsconfig.json to Dockerfile COPY step (required by tsc --build)
  • Use npm install instead of npm ci since package-lock.json is not tracked in the repo
  • Move Docker instructions from Quick Start into a collapsible <details> section
  • Quick Start now leads with the install script + tinyagi command

Testing

  • Verified docker compose build succeeds
  • Verified docker compose up -d starts the container

Checklist

  • PR title follows conventional commit format (type(scope): description)
  • I have tested these changes locally
  • My changes don't introduce new warnings or errors
  • I have updated documentation if needed

jlia0 and others added 3 commits March 26, 2026 07:23
…in README

- Skip Puppeteer browser download in builder stage (system Chromium used at runtime)
- Add missing tsconfig.json to COPY step for tsc --build
- Use npm install instead of npm ci (no package-lock.json in repo)
- Move Docker instructions from Quick Start into a collapsible details section

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Create non-root tinyagi user to fix Claude CLI root restriction
- Write settings.json with Docker-appropriate paths (/data/workspace)
- Bootstrap agent working directory with templates
- Symlink tinyagi CLI to /usr/local/bin
- Install gosu for root-to-user switching

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The entrypoint writes tinyagi.pid before exec-ing node, so the CLI's
status/stop commands correctly detect the running process.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jlia0 jlia0 merged commit 363e2cf into main Mar 26, 2026
@jlia0 jlia0 deleted the jlia0/fix-readme-docker branch March 26, 2026 00:04
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