Skip to content

fix(core): resolve .dockerignore validation bug in workspace discovery#5

Merged
PunGrumpy merged 4 commits into
mainfrom
advisor/001-fix-dockerignore-validation
Jul 11, 2026
Merged

fix(core): resolve .dockerignore validation bug in workspace discovery#5
PunGrumpy merged 4 commits into
mainfrom
advisor/001-fix-dockerignore-validation

Conversation

@PunGrumpy

Copy link
Copy Markdown
Owner

This PR resolves Plan 001: Fix the .dockerignore validation bug in workspace discovery.

Summary

  1. Workspace Discovery: Updated discoverProject to detect and track .dockerignore files under a new dockerignores property in the ProjectInfo return value.
  2. Type Safety: Updated the ProjectInfo type definition in packages/core/src/types/index.ts to include the optional dockerignores field.
  3. CLI Orchestration: Merged the discovered .dockerignore paths into the projectFilesList passed to the rule runner engine.
  4. Complexity Reduction: Extracted the category filtering logic in the Commander CLI action handler into a clean, top-level filterDiagnostics helper function to keep complexity under 20.

All typechecks, builds, tests, and formatting checks pass cleanly.

@vercel

vercel Bot commented Jul 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docker-doctor Ready Ready Preview, Comment Jul 11, 2026 5:46pm

@PunGrumpy PunGrumpy merged commit 0a04d98 into main Jul 11, 2026
9 checks passed
@PunGrumpy PunGrumpy deleted the advisor/001-fix-dockerignore-validation branch July 11, 2026 17:51
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