Skip to content

fix: add missing gitignore files for cua templates#144

Merged
masnwilliams merged 1 commit intohypeship/unified-cua-templatefrom
ci-fix/hypeship/unified-cua-template
Apr 8, 2026
Merged

fix: add missing gitignore files for cua templates#144
masnwilliams merged 1 commit intohypeship/unified-cua-templatefrom
ci-fix/hypeship/unified-cua-template

Conversation

@masnwilliams
Copy link
Copy Markdown
Contributor

@masnwilliams masnwilliams commented Apr 8, 2026

Add the standard template _gitignore files for the new TypeScript and Python CUA templates so generated apps include .gitignore and the template tests pass again.

Made-with: Cursor


Note

Low Risk
Low risk because it only adds template ignore patterns and does not affect runtime logic or production behavior.

Overview
Adds _gitignore files to the Python and TypeScript cua templates so generated apps include a .gitignore with standard ignores (e.g., Python virtualenv/cache folders and TypeScript node_modules/bun.lockb), restoring expected template/test behavior.

Reviewed by Cursor Bugbot for commit e46a020. Bugbot is set up for automated code reviews on this repo. Configure here.

Add the standard template _gitignore files for the new TypeScript and Python CUA templates so generated apps include .gitignore and the template tests pass again.

Made-with: Cursor
@masnwilliams masnwilliams merged commit 99891de into hypeship/unified-cua-template Apr 8, 2026
@masnwilliams masnwilliams deleted the ci-fix/hypeship/unified-cua-template branch April 8, 2026 20:09
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e46a020. Configure here.

@@ -0,0 +1,2 @@
node_modules
bun.lockb
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

TypeScript CUA gitignore missing .env pattern

High Severity

The TypeScript CUA _gitignore is missing the .env pattern, unlike every other TypeScript template in the repo. The CUA template's own README.md instructs users to cp .env.example .env and store API keys (ANTHROPIC_API_KEY, OPENAI_API_KEY, GOOGLE_API_KEY) there. Without .env in the gitignore, users are likely to accidentally commit secrets to version control. The Python CUA _gitignore correctly includes .env, making this an oversight specific to the TypeScript version.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e46a020. Configure here.

@@ -0,0 +1,2 @@
node_modules
bun.lockb
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Both CUA gitignore files missing standard patterns

Medium Severity

Both CUA _gitignore files are dramatically smaller than every other template's gitignore (2–4 lines vs. 39–74 lines). Standard patterns present in all sibling templates — like IDE files (.idea/, .vscode/), OS artifacts (.DS_Store), build output (dist/, build/), log files (*.log), and testing artifacts — are entirely absent. The PR description says these are "the standard template _gitignore files," but they are far from standard compared to every other template in the repo.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e46a020. Configure here.

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