feat: open channel previews from QR deeplinks#2305
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds channel-preview deep-link contracts, registers native URL schemes and associated domains, installs a native deep-link handler, adds a /preview/channel landing route, extends the scanner to process preview links and start preview sessions, builds channel-specific QR deep links, and threads preview mode through backend queries, validation, and tests. ChangesChannel preview deep-linking and QR support
Sequence Diagram(s)sequenceDiagram
participant OS as Operating System
participant DeepLinkService as src/services/deepLinks.ts
participant Router as Vue Router
participant Scanner as src/pages/scan.vue
OS->>DeepLinkService: appUrlOpen event or launch URL
alt Preview deep-link
DeepLinkService->>Router: navigate /scan?preview=URL
Router->>Scanner: route with preview param
else Console HTTPS
DeepLinkService->>Router: navigate original pathname+search+hash
end
sequenceDiagram
participant Scanner as src/pages/scan.vue
participant Parser as src/services/previewLinks.ts
participant Updater as CapacitorUpdater
participant API as Capgo API
Scanner->>Scanner: check route.query.preview
alt Preview param exists
Scanner->>Parser: parseChannelPreviewDeepLink(url)
Parser-->>Scanner: ChannelPreviewLink
Scanner->>API: getLatest({ preview: true }) for channel
API-->>Scanner: bundle metadata
Scanner->>Updater: download & set(bundle)
Updater-->>Scanner: applied
Scanner->>Updater: startPreviewSession()
else No preview param
Scanner->>Scanner: start barcode scanner
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (1)
src/services/deepLinks.ts (1)
4-4: ⚡ Quick winUse
~/alias instead of relative import insrc/code.Please switch this import to the project alias to match frontend import conventions.
Suggested fix
-import { parseChannelPreviewDeepLink } from './previewLinks' +import { parseChannelPreviewDeepLink } from '~/services/previewLinks'As per coding guidelines: "Import using
~/alias forsrc/directory in frontend code instead of relative paths".🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/services/deepLinks.ts` at line 4, The import in deepLinks.ts currently uses a relative path for parseChannelPreviewDeepLink; update the import to use the project alias for src by replacing the relative import ("./previewLinks") with the alias path ("~/services/previewLinks" or whichever alias maps to the previewLinks module) so parseChannelPreviewDeepLink is imported via the ~/ root import convention consistent with frontend code.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@android/app/src/main/AndroidManifest.xml`:
- Around line 28-30: The Android app-link intent filter is missing the staging
host declared by isCapgoConsoleHost(); add a new <data android:scheme="https"
android:host="console.staging.capgo.app" /> entry alongside the existing
console.capgo.app, console.dev.capgo.app, and console.preprod.capgo.app <data>
elements inside the app's intent-filter in AndroidManifest.xml so HTTPS deep
links for console.staging.capgo.app will be handed off to the app.
In `@ios/App/App/App.entitlements`:
- Around line 9-12: Add the missing staging associated-domain entry by inserting
the applinks domain string for staging (applinks:console.staging.capgo.app)
alongside the existing applinks entries (e.g., applinks:console.capgo.app,
applinks:console.dev.capgo.app, applinks:console.preprod.capgo.app) in the
App.entitlements so iOS universal links from staging will open the app; ensure
the new string follows the same XML <string>…</string> pattern as the others.
In `@src/pages/scan.vue`:
- Around line 97-103: The code sets the preview channel via
CapacitorUpdater.next() before confirming the preview will work; defer changing
the channel until after startPreviewSession()/getLatest() and any download
succeed, or if you must change it earlier, capture the original channel (via
CapacitorUpdater.current()/current.bundle.id) and ensure you call
CapacitorUpdater.setChannel()/CapacitorUpdater.next() to restore that original
channel in every error path (catch blocks) and before exiting the fallback
branch; update the logic around startPreviewSession(), getLatest(),
CapacitorUpdater.next(), and the error handlers so the preview channel is only
persisted on successful start and is restored on failure.
---
Nitpick comments:
In `@src/services/deepLinks.ts`:
- Line 4: The import in deepLinks.ts currently uses a relative path for
parseChannelPreviewDeepLink; update the import to use the project alias for src
by replacing the relative import ("./previewLinks") with the alias path
("~/services/previewLinks" or whichever alias maps to the previewLinks module)
so parseChannelPreviewDeepLink is imported via the ~/ root import convention
consistent with frontend code.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 64b9922e-537d-44de-85e6-07f50db563b1
📒 Files selected for processing (11)
android/app/src/main/AndroidManifest.xmlios/App/App/App.entitlementsios/App/App/Info.plistsrc/components/BundlePreviewFrame.vuesrc/main.tssrc/pages/app/[app].channel.[channel].preview.vuesrc/pages/preview/channel.vuesrc/pages/scan.vuesrc/route-map.d.tssrc/services/deepLinks.tssrc/services/previewLinks.ts
riderx
force-pushed
the
codex/preview-channel-deeplink
branch
from
4e2111a to
e436e42
Compare
Merging this PR will not alter performance
Comparing 
Footnotes
|
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/updates.test.ts`:
- Line 587: Change the two new tests from synchronous `it()` to parallel
`it.concurrent()` to follow the repo's parallel-test convention: locate the test
with the description "allows preview requests to fetch a private target channel
when app preview is enabled" and replace `it(` with `it.concurrent(`, and do the
same for the other new test referenced in the comment (around line 646) so both
run concurrently within the same file.
- Around line 618-643: The test flips shared app state by setting allow_preview
to true then later resetting it; wrap the mutable section (the call that sets
allow_preview true, the setup of baseData, the call to postUpdate and assertions
using postUpdate and parseSchema/updateNewScheme) in a try/finally so the final
supabase update that sets allow_preview back to false always runs even if
assertions throw; locate the two supabase.update calls referencing
APP_NAME_UPDATE and move the second one into a finally block, keeping
throwOnError on both updates to surface errors.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 883969bd-2731-44e8-bd42-df73dc42ebbf
📒 Files selected for processing (10)
capacitor.config.tssrc/pages/scan.vuesupabase/functions/_backend/utils/pg.tssupabase/functions/_backend/utils/plugin_parser.tssupabase/functions/_backend/utils/plugin_validation.tssupabase/functions/_backend/utils/postgres_schema.tssupabase/functions/_backend/utils/types.tssupabase/functions/_backend/utils/update.tstests/test-utils.tstests/updates.test.ts
achimala
left a comment
achimala
left a comment
There was a problem hiding this comment.
This branch depends on updater APIs that the app does not actually ship yet. src/pages/scan.vue calls getLatest({ appId, channel }) and startPreviewSession({ appId }), but bun.lock still resolves @capgo/capacitor-updater to 8.45.11.
In 8.45.11, GetLatestOptions only contains channel, Android/iOS getLatest only read that channel, and there is no native startPreviewSession plugin method registered. The local type cast masks this from TypeScript, but the QR flow will run against the installed native plugin: the target app id will be ignored, and devices can fall back to the old current()/next()/setShakeMenu() branch instead of starting a real cross-app preview session.
Please land/release the updater-side preview APIs and bump @capgo/capacitor-updater plus bun.lock here before merging this app-side QR flow. At minimum, this should have a version guard or native smoke test so the scanner cannot silently advertise cross-app previews while the bundled native plugin is still on the old API surface.
hadessunxy-code
left a comment
hadessunxy-code
left a comment
There was a problem hiding this comment.
Reviewed the QR channel preview flow and found one blocker in the native scan path.
|
I checked the latest scan flow after the updater bump. Risk: if the download or Verification: |
riderx
force-pushed
the
codex/preview-channel-deeplink
branch
from
8d62ea5 to
1d5c20a
Compare
…-deeplink # Conflicts: # supabase/migrations/20260528090340_manifest_index.sql
|
Reviewed current head 5dfdb29 with focus on the QR preview deep-link contract and the native scan entry path. I also checked that the earlier updater API compatibility concern is no longer present in this branch: package.json/bun.lock now resolve @capgo/capacitor-updater to 8.47.4, and the scan page calls the typed startPreviewSession/getLatest preview APIs instead of relying on the older fallback path for payload-backed links. Local verification from this branch:
The parser/builders cover payload-backed channel links, legacy web channel links, native capgo:/ variations, and bundle payload links. In scan.vue, payload-backed links go through startPreviewPayload, while the getLatest({ preview: true }) path is kept for legacy channel links without a payload URL. No blocker from this focused pass. I did not do the physical-device production scan listed under Not Tested. |
|
6 participants
What
Why
How
capgo://preview/channel?...&url=<preview.json>.capgo://preview/bundle?url=<preview.json>./.capgo/preview.json; no update backend or channel-assignment backend code is changed.Testing
bun test tests/preview-links.unit.test.tsbun run typecheck:frontendbun run lintbun test tests/preview-links.unit.test.ts && bun run buildbunx cap copyxcodebuild -workspace ios/App/App.xcworkspace -scheme App -configuration Debug -destination generic/platform=iOS CODE_SIGNING_ALLOWED=NONot Tested