Skip to content

fix(test): run ctt with TMPDIR=/tmp - #65

Merged
tsvikas merged 1 commit into
mainfrom
fix-ci-lints
Sep 27, 2026
Merged

tsvikas merged 1 commit into
mainfrom
fix-ci-lints

Conversation

@tsvikas

@tsvikas tsvikas commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

just test failed locally on macOS in every ctt case, at the detect-secrets step of the generated just init.

Cause

ctt (2.5.0 and 3.0.0 alike) renders from a copy under $TMPDIR and only rewrites _src_path in .copier-answers.yml after the _post_tasks have run. On macOS, $TMPDIR is /var/folders/xx/<random>/T/, and detect-secrets flags that path as a Base64 High Entropy String. On Linux the path is /tmp/ctt-src-xxxx, so CI never hit this.

Change

justfile: test runs TMPDIR=/tmp uv run ctt, with a comment explaining why. Updating copier (9.18.2), ctt (3.0.0) and detect-secrets (1.5.0) did not fix it; the dependency update is in a separate PR.

Verification

  • Before: uv run ctt -t bare fails at detect-secrets with the default macOS TMPDIR, on both ctt 3.0.0 and 2.5.0.
  • After: just test passes bare. The next case, docs, fails only on zizmor's new self-repository audit, which is a separate PR.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GyMoD8tx4oz1bpJ1tEL845

@sourcery-ai sourcery-ai 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.

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@tsvikas tsvikas changed the title fix(test): run ctt with TMPDIR=/tmp; update locked deps fix(test): run ctt with TMPDIR=/tmp Sep 27, 2026
ctt renders from a copy under $TMPDIR, and the generated `just init` runs
detect-secrets before ctt normalizes `_src_path` in .copier-answers.yml.
On macOS the per-user $TMPDIR (/var/folders/xx/<random>/T) reads as a
high-entropy secret, so `just test` failed locally. Linux CI was fine.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01GyMoD8tx4oz1bpJ1tEL845
@tsvikas
tsvikas merged commit 7d183da into main Sep 27, 2026
5 checks passed
@tsvikas
tsvikas deleted the fix-ci-lints branch September 27, 2026 12:17
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