Skip to content

feat: expand skipDirs to 24 entries, sync across all slices #41

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jonathanpopham merged 1 commit into from
Apr 3, 2026
Merged

feat: expand skipDirs to 24 entries, sync across all slices #41

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 24 additions & 14 deletions internal/analyze/zip.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,20 +12,30 @@ import (

// skipDirs are directory names that should never be included in the archive.
var skipDirs = map[string]bool{
".git": true,
"node_modules": true,
"vendor": true,
"__pycache__": true,
".venv": true,
"venv": true,
"dist": true,
"build": true,
"target": true,
".next": true,
".nuxt": true,
"coverage": true,
".terraform": true,
".tox": true,
".git": true,
".claude": true,
".idea": true,
".vscode": true,
".cache": true,
".turbo": true,
".nx": true,
".next": true,
".nuxt": true,
".terraform": true,
".tox": true,
".venv": true,
".pnpm-store": true,
"__pycache__": true,
"__snapshots__": true,
"bower_components": true,
"build": true,
"coverage": true,
"dist": true,
"node_modules": true,
"out": true,
"target": true,
"vendor": true,
"venv": true,
}

// createZip archives the repository at dir into a temporary ZIP file and
Expand Down
38 changes: 24 additions & 14 deletions internal/archdocs/zip.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,20 +12,30 @@ import (

// skipDirs are directory names that should never be included in the archive.
var skipDirs = map[string]bool{
".git": true,
"node_modules": true,
"vendor": true,
"__pycache__": true,
".venv": true,
"venv": true,
"dist": true,
"build": true,
"target": true,
".next": true,
".nuxt": true,
"coverage": true,
".terraform": true,
".tox": true,
".git": true,
".claude": true,
".idea": true,
".vscode": true,
".cache": true,
".turbo": true,
".nx": true,
".next": true,
".nuxt": true,
".terraform": true,
".tox": true,
".venv": true,
".pnpm-store": true,
"__pycache__": true,
"__snapshots__": true,
"bower_components": true,
"build": true,
"coverage": true,
"dist": true,
"node_modules": true,
"out": true,
"target": true,
"vendor": true,
"venv": true,
}

// createZip archives the repository at dir into a temporary ZIP file and
Expand Down
38 changes: 24 additions & 14 deletions internal/blastradius/zip.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,20 +12,30 @@ import (

// skipDirs are directory names that should never be included in the archive.
var skipDirs = map[string]bool{
".git": true,
"node_modules": true,
"vendor": true,
"__pycache__": true,
".venv": true,
"venv": true,
"dist": true,
"build": true,
"target": true,
".next": true,
".nuxt": true,
"coverage": true,
".terraform": true,
".tox": true,
".git": true,
".claude": true,
".idea": true,
".vscode": true,
".cache": true,
".turbo": true,
".nx": true,
".next": true,
".nuxt": true,
".terraform": true,
".tox": true,
".venv": true,
".pnpm-store": true,
"__pycache__": true,
"__snapshots__": true,
"bower_components": true,
"build": true,
"coverage": true,
"dist": true,
"node_modules": true,
"out": true,
"target": true,
"vendor": true,
"venv": true,
}

// createZip archives the repository at dir into a temporary ZIP file and
Expand Down
38 changes: 24 additions & 14 deletions internal/deadcode/zip.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,20 +12,30 @@ import (

// skipDirs are directory names that should never be included in the archive.
var skipDirs = map[string]bool{
".git": true,
"node_modules": true,
"vendor": true,
"__pycache__": true,
".venv": true,
"venv": true,
"dist": true,
"build": true,
"target": true,
".next": true,
".nuxt": true,
"coverage": true,
".terraform": true,
".tox": true,
".git": true,
".claude": true,
".idea": true,
".vscode": true,
".cache": true,
".turbo": true,
".nx": true,
".next": true,
".nuxt": true,
".terraform": true,
".tox": true,
".venv": true,
".pnpm-store": true,
"__pycache__": true,
"__snapshots__": true,
"bower_components": true,
"build": true,
"coverage": true,
"dist": true,
"node_modules": true,
"out": true,
"target": true,
"vendor": true,
"venv": true,
}

// createZip archives the repository at dir into a temporary ZIP file and
Expand Down
38 changes: 24 additions & 14 deletions internal/factory/zip.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,20 +11,30 @@ import (

// skipDirs are directory names that should never be included in the archive.
var skipDirs = map[string]bool{
".git": true,
"node_modules": true,
"vendor": true,
"__pycache__": true,
".venv": true,
"venv": true,
"dist": true,
"build": true,
"target": true,
".next": true,
".nuxt": true,
"coverage": true,
".terraform": true,
".tox": true,
".git": true,
".claude": true,
".idea": true,
".vscode": true,
".cache": true,
".turbo": true,
".nx": true,
".next": true,
".nuxt": true,
".terraform": true,
".tox": true,
".venv": true,
".pnpm-store": true,
"__pycache__": true,
"__snapshots__": true,
"bower_components": true,
"build": true,
"coverage": true,
"dist": true,
"node_modules": true,
"out": true,
"target": true,
"vendor": true,
"venv": true,
}

// CreateZip archives the repository at dir into a temporary ZIP file and
Expand Down
51 changes: 42 additions & 9 deletions internal/find/zip.go
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
package find

// zip.go duplicates archive helpers to preserve vertical slice isolation.

import (
"archive/zip"
"fmt"
Expand All @@ -12,25 +10,53 @@ import (
"strings"
)

// skipDirs are directory names that should never be included in the archive.
var skipDirs = map[string]bool{
".git": true, "node_modules": true, "vendor": true,
"__pycache__": true, ".venv": true, "venv": true,
"dist": true, "build": true, "target": true,
".next": true, ".terraform": true,
".git": true,
".claude": true,
".idea": true,
".vscode": true,
".cache": true,
".turbo": true,
".nx": true,
".next": true,
".nuxt": true,
".terraform": true,
".tox": true,
".venv": true,
".pnpm-store": true,
"__pycache__": true,
"__snapshots__": true,
"bower_components": true,
"build": true,
"coverage": true,
"dist": true,
"node_modules": true,
"out": true,
"target": true,
"vendor": true,
"venv": true,
}

// createZip archives the repository at dir into a temporary ZIP file and
// returns its path. The caller is responsible for removing the file.
//
// Strategy: use git archive when inside a Git repo (respects .gitignore,
// deterministic output). Falls back to a manual directory walk otherwise.
func createZip(dir string) (string, error) {
f, err := os.CreateTemp("", "supermodel-find-*.zip")
f, err := os.CreateTemp("", "supermodel-*.zip")
if err != nil {
return "", fmt.Errorf("create temp: %w", err)
return "", fmt.Errorf("create temp file: %w", err)
}
dest := f.Name()
f.Close()

if isGitRepo(dir) {
if err := gitArchive(dir, dest); err == nil {
return dest, nil
}
}

if err := walkZip(dir, dest); err != nil {
os.Remove(dest)
return "", err
Expand All @@ -51,19 +77,26 @@ func gitArchive(dir, dest string) error {
return cmd.Run()
}

// walkZip creates a ZIP of dir, excluding skipDirs, hidden files, and
// files larger than 10 MB.
func walkZip(dir, dest string) error {
out, err := os.Create(dest)
if err != nil {
return err
}
defer out.Close()

zw := zip.NewWriter(out)
defer zw.Close()

return filepath.Walk(dir, func(path string, info os.FileInfo, err error) error {
if err != nil {
return err
}
rel, _ := filepath.Rel(dir, path)
rel, err := filepath.Rel(dir, path)
if err != nil {
return err
}
if info.IsDir() {
if skipDirs[info.Name()] {
return filepath.SkipDir
Expand Down
Loading
Loading