Skip to content

feat(worktree): support copy patterns#99

Merged
nettee merged 6 commits into
mainfrom
copy-config
Jun 25, 2026
Merged

feat(worktree): support copy patterns#99
nettee merged 6 commits into
mainfrom
copy-config

Conversation

@nettee

@nettee nettee commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace split copy config with global-only top-level copy = [...] Copy Patterns
  • Match Git-ignored files/symlinks with gitignore-style patterns, dedupe overlaps, and summarize copy output
  • Update default config template, docs, e2e coverage, and mark the active Zest spec implemented

Validation

  • cargo fmt && cargo check && cargo test
  • uv run --project e2e pytest e2e/test_env_copy.py
  • uv run --project e2e pytest e2e/test_repo_mode.py e2e/test_auxiliary_group.py e2e/test_pnpm.py
  • scripts/test-install-local.sh && scripts/test-install.sh

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b83451f6fa

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/worktree.rs Outdated
Comment thread src/worktree.rs
Constrain ignored-file enumeration with pattern-derived pathspecs and treat slashless copy patterns as directory basenames for ignored descendants.

Generated-By: looper 0.9.11 (runner=fixer, agent=codex)
@nettee

nettee commented Jun 25, 2026

Copy link
Copy Markdown
Owner Author

Looper fixer round complete508f0b2

  • ✅ Review comment on src/worktree.rs (@chatgpt-codex-connector) — thread
    • Updated src/worktree.rs so copy_pattern_ignored_files passes pattern-derived git glob pathspecs to git ls-files, which constrains ignored-file enumeration to the configured copy patterns instead of scanning the full ignored tree.
  • ✅ Review comment on src/worktree.rs:1758 (@chatgpt-codex-connector) — thread
    • Expanded slashless copy-pattern matching in src/worktree.rs so basename patterns also match ignored descendants under directories like secrets/, and added regression coverage in e2e/test_env_copy.py plus focused unit tests.

🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 508f0b260b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/worktree.rs
Generated-By: looper 0.9.11 (runner=fixer, agent=codex)
@nettee

nettee commented Jun 25, 2026

Copy link
Copy Markdown
Owner Author

Looper fixer round completee2fcd61

  • ✅ Review comment on src/worktree.rs:1755 (@chatgpt-codex-connector) — thread
    • Updated src/worktree.rs so copy-pattern descendant matching also covers slash-containing patterns without a trailing slash, and added regression coverage in src/worktree.rs and e2e/test_env_copy.py for copy = ["specs/change/active"].

🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e2fcd61a7d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/worktree.rs
Generated-By: looper 0.9.11 (runner=fixer, agent=codex)
@nettee

nettee commented Jun 25, 2026

Copy link
Copy Markdown
Owner Author

Looper fixer round complete011f8c2

  • ✅ Review comment on src/worktree.rs:1795 (@chatgpt-codex-connector) — thread
    • Updated src/worktree.rs so trailing-slash copy patterns generate nested descendant pathspecs and matcher coverage at any depth, then added Rust and e2e regression tests for nested .agents/ directories.

🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 011f8c2985

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/worktree.rs
Generated-By: looper 0.9.11 (runner=fixer, agent=codex)
@nettee

nettee commented Jun 25, 2026

Copy link
Copy Markdown
Owner Author

Looper fixer round completeb5a1db4

  • ✅ Review comment on src/worktree.rs:1523 (@chatgpt-codex-connector) — thread
    • Updated src/worktree.rs to fail fast when a copy pattern starts with !, added a unit test and an e2e regression for that validation failure, and clarified the limitation in docs/user-guide.md.

🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@nettee nettee merged commit 7809b7c into main Jun 25, 2026
2 checks passed
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