Skip to content

fix(tooling): ignore vinxi timestamp sidecars in .gitignore - #2927

Merged
richiemcilroy merged 2 commits into
CapSoftware:mainfrom
ghazy2001:fix/vinxi-timestamp-sidecar-gitignore-2923
Oct 23, 2026
Merged

richiemcilroy merged 2 commits into
CapSoftware:mainfrom
ghazy2001:fix/vinxi-timestamp-sidecar-gitignore-2923

Conversation

@ghazy2001

@ghazy2001 ghazy2001 commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Resolves #2923.

When builds using Vinxi (e.g. apps/desktop) are cancelled or interrupted by SIGINT before normal cleanup, Vinxi's temporary config bundle (apps/desktop/app.config.timestamp_<ms>.js generated by vinxi/lib/load-app.js) is left behind untracked in the working tree.

This adds *.config.timestamp_*.js to root .gitignore directly under .vinxi to cleanly prevent workspace pollution.

Verification

Verified with git check-ignore:

$ git check-ignore -v apps/desktop/app.config.timestamp_1792674498991.js
.gitignore:41:*.config.timestamp_*.js	apps/desktop/app.config.timestamp_1792674498991.js

Fixes #2923

RetriggerView in GreptileConfidence Score: 5/5

This PR appears safe to merge.

Summary

Adds *.config.timestamp_*.js to the root .gitignore so temporary Vinxi config files stay out of Git's untracked-file list.

  • Git ignores timestamped JavaScript config bundles across the workspace.

Reviews (1) · Last reviewed commit: "fix(tooling): ignore vinxi timestamp sid..." · Reviewed by Greptile

Add *.config.timestamp_*.js to root .gitignore to prevent untracked sidecars created by vinxi load-app during interrupted builds from polluting working directory.

Fixes CapSoftware#2923
@vercel

vercel Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

@ghazy2001 is attempting to deploy a commit to the Cap Software Inc Team on Vercel.

A member of the Team first needs to authorize it.

@richiemcilroy
richiemcilroy merged commit 9ec74e1 into CapSoftware:main Oct 23, 2026
43 of 45 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.

Tooling: interrupted builds leave untracked apps/desktop/app.config.timestamp_*.js (not gitignored)

2 participants