Skip to content

chore: manage development services with Compose - #700

Open
kylos101 wants to merge 4 commits into
openai:mainfrom
kylos101:kb/devcontainer-automation
Open

chore: manage development services with Compose#700
kylos101 wants to merge 4 commits into
openai:mainfrom
kylos101:kb/devcontainer-automation

Conversation

@kylos101

@kylos101 kylos101 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Manage the scanner and findings service through the Docker Compose engine inside the existing Universal Dev Container, without adding a Dev Container Feature.

Changes

  • Add the findings service to the root Compose project and require host networking for both application services.
  • Build both application images from the install task.
  • Add a manual and post-start findings task that depends on install and starts findings through Compose.
  • Ignore the generated .pnpm-store/.

Testing

# Validate the Dev Container and Ona automation configuration.
gitpod environment devcontainer validate .devcontainer/devcontainer.json
gitpod environment config validate .ona/automations.yml

# Stop all Compose containers and remove both service images.
docker compose --profile scan down --rmi all

# Build both service images.
docker compose build codex-security findings

# Start Compose and manually verify its default services.
docker compose up

# Press Ctrl+C, then stop Compose and all services.
docker compose down

Risk and rollout

The findings service claims host port 3000 when the Dev Container starts. Scanner execution remains manual and retains its existing user, seccomp, capability, credential, and filesystem protections. Removing volumes permanently deletes local findings data. No public CLI surface changes.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

kylos101 and others added 4 commits August 28, 2026 02:22
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@kylos101
kylos101 marked this pull request as ready for review August 28, 2026 03:03
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.

2 participants