refactor(choice-builder): deduplicate canvas target path check

[chhoumann]

Use the existing isCanvasTargetPath(...) helper in CaptureChoiceBuilder where canvas target detection was previously duplicated inline.

This keeps the .canvas detection logic in one place in the class and reduces risk of subtle drift if the check changes in the future. Behavior is unchanged.

Alternative considered: leave the duplication in place due its small size, but centralizing the check is clearer and lower maintenance.

Reviewer context: this PR is intentionally tiny (1-line functional change) and limited to addCanvasModeCompatibilityNotice(...) in the choice builder.

---

Summary by CodeRabbit

• Refactor
  • Optimized internal code organization and logic flow to enhance maintainability and code consistency. These improvements strengthen the overall codebase quality while ensuring all existing features and functionality remain fully intact. The modifications focus exclusively on code quality enhancements with no impact on end-user experience, application behavior, or any user-facing features.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
quickadd	Ready	Preview	Mar 12, 2026 6:11am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: adfbc2f4-9aa9-411e-ba3c-caca09c99a09

📥 Commits

Reviewing files that changed from the base of the PR and between 5e5cd69 and c8f699d.

📒 Files selected for processing (1)

• src/gui/ChoiceBuilder/captureChoiceBuilder.ts

---

📝 Walkthrough

Walkthrough

A refactoring change extracts a canvas target path validation check into a dedicated helper method (isCanvasTargetPath), replacing an inline string operation that trims, lowercases, and checks if a path ends with ".canvas".

Changes

Cohort / File(s)	Summary
Canvas Target Path Validation src/gui/ChoiceBuilder/captureChoiceBuilder.ts	Replaced direct .canvas extension check with isCanvasTargetPath() helper method call. Changes source from this.choice.captureTo.trim().toLowerCase().endsWith(".canvas") to delegated method invocation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A helper method hops into view,
Catching canvas paths, shiny and new,
Logic extracted, clean and refined,
Refactored with care, a beautiful find!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: replacing an inline canvas target check with the existing isCanvasTargetPath helper method to deduplicate logic.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch codex/small-targeted-refactor-canvas-target-check

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cloudflare-workers-and-pages]

Deploying quickadd with    Cloudflare Pages

Latest commit:	c8f699d
Status:	✅  Deploy successful!
Preview URL:	https://3f629891.quickadd.pages.dev
Branch Preview URL:	https://codex-small-targeted-refacto-z462.quickadd.pages.dev

View logs

[devin-ai-integration]

✅ Devin Review: No Issues Found

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

View in Devin Review to see 1 additional finding.