Skip to content

fix(docker): scan Dockerfile variants and every Compose file - #600

Open
vibgrate-team wants to merge 1 commit into
mainfrom
cursor/docker-manifest-names-0d82
Open

vibgrate-team wants to merge 1 commit into
mainfrom
cursor/docker-manifest-names-0d82

Conversation

@vibgrate-team

Copy link
Copy Markdown
Contributor

Summary

vg scan only treated five exact filenames as Docker manifests (Dockerfile, docker-compose.yml, docker-compose.yaml, compose.yml, compose.yaml), and it stopped after the first Compose file in a directory. Common layouts were skipped.

The scanner now recognises:

  • Dockerfile, Containerfile, Dockerfile.<suffix> (Dockerfile.prod), and <name>.Dockerfile (api.Dockerfile)
  • docker-compose.yml / .yaml, compose.yml / .yaml, and overrides docker-compose.<name>.yml / .yaml and compose.<name>.yml / .yaml

Every matching file in a directory is parsed. Image rows stay deduplicated on name and tag, so the same image and tag in two files is one row and two tags stay two rows. The recognised names are listed in DOCS.md.

Related issues

Fixes #580

Checklist

  • pnpm test passes
  • pnpm lint is clean
  • pnpm typecheck is clean
  • Docs updated (README / DOCS / ARCHITECTURE) where behavior changed
  • Determinism preserved — identical input still produces identical graph.json / report output (content-hashed IDs, stable sorts; no time, randomness, or filesystem-order dependence)
  • No proprietary or internal references — public, Apache-2.0 content only
  • Commits use Conventional Commits and are signed off (git commit -s, DCO)

Notes for reviewers

Project directories are sorted by path before they are scanned, so a tree with several Docker projects reports them in a stable order. Names are matched with the capitalisation above (Containerfile, not containerfile).

Open in Web Open in Cursor 

Recognise Containerfile, Dockerfile.<suffix>, and <name>.Dockerfile, and
parse every Compose file in a directory, including overrides. Image rows
stay deduplicated on name and tag.

Fixes #580

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: vibgrate-team <vibgrate-team@users.noreply.github.com>
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.

Enhancement: scan Dockerfile.*, *.Dockerfile, Containerfile and every Compose file in a directory

2 participants