Skip to content

Exclude Capacitor sync output from codebase context#3220

Open
riderx wants to merge 1 commit intodyad-sh:mainfrom
riderx:codex/exclude-capacitor-sync-output
Open

Exclude Capacitor sync output from codebase context#3220
riderx wants to merge 1 commit intodyad-sh:mainfrom
riderx:codex/exclude-capacitor-sync-output

Conversation

@riderx
Copy link
Copy Markdown

@riderx riderx commented Apr 15, 2026

Summary

  • exclude Capacitor-synced web bundle copies from codebase extraction
  • keep actual Android and iOS source files available in context
  • add regression coverage for both native Git and isomorphic-git collection paths

Why

Capacitor sync copies the built web app into native folders like android/app/src/main/assets/public and ios/App/App/public. Those files are generated artifacts and can drastically inflate token usage in Dyad after mobile syncs.

Addresses #1149.

Testing

  • npm test -- src/tests/git_utils.test.ts src/tests/codebase.test.ts
  • npm run lint
  • npm run fmt:check
  • npm run ts

@github-actions
Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a mechanism to exclude Capacitor-generated artifacts from the codebase extraction process to prevent token inflation. It defines specific path prefixes for Android and iOS Capacitor build outputs and implements path normalization and exclusion logic in both native and isomorphic git collection methods. Additionally, a new test suite was added to verify these exclusions. I have no feedback to provide.

@riderx riderx changed the title [codex] Exclude Capacitor sync output from codebase context Exclude Capacitor sync output from codebase context Apr 15, 2026
@riderx riderx marked this pull request as ready for review April 15, 2026 12:29
@riderx riderx requested a review from a team April 15, 2026 12:29
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

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