Skip to content

fix: remove unused @ts-expect-error directive in scan.vue#1363

Merged
riderx merged 2 commits into
feature/domain-based-auto-joinfrom
copilot/fix-action-step-error
Jan 8, 2026
Merged

fix: remove unused @ts-expect-error directive in scan.vue#1363
riderx merged 2 commits into
feature/domain-based-auto-joinfrom
copilot/fix-action-step-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 5, 2026

Summary

Removed unused @ts-expect-error directive from src/pages/scan.vue that was causing TypeScript compilation to fail. The directive was suppressing a non-existent type error for the barcode scanner import.

// Before (TypeScript error: Unused '@ts-expect-error' directive)
// @ts-expect-error - barcode scanner module may not be available in all environments
import { CapacitorBarcodeScanner } from '@capacitor/barcode-scanner'

// After
import { CapacitorBarcodeScanner } from '@capacitor/barcode-scanner'

Test plan

  1. Run bun typecheck - verify the "Unused '@ts-expect-error' directive" error is resolved
  2. Run bun lint && bun lint:backend - verify all linting passes
  3. Build the project - verify no compilation errors

Screenshots

N/A - build/type checking fix only

Checklist

  • My code follows the code style of this project and passes bun run lint:backend && bun run lint.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • My change has adequate E2E test coverage.
  • I have tested my code manually, and I have provided steps how to reproduce my tests

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • npm.jsr.io
    • Triggering command: /usr/local/bin/bun bun install (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Reference: https://github.com/Cap-go/capgo/actions/runs/20701865047/job/59425399658#step:10:1


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Note

Removes an unused @ts-expect-error directive from src/pages/scan.vue that was suppressing a non-existent error on CapacitorBarcodeScanner import, fixing TypeScript typecheck.

  • Deleted the @ts-expect-error comment above the @capacitor/barcode-scanner import
  • No functional runtime changes; build/typechecking only

Written by Cursor Bugbot for commit e6a37bf. This will update automatically on new commits. Configure here.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 5, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


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

Co-authored-by: jokabuyasina <2309056+jokabuyasina@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix action step error in Capgo workflow fix: remove unused @ts-expect-error directive in scan.vue Jan 5, 2026
Copilot AI requested a review from jokabuyasina January 5, 2026 01:22
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jan 5, 2026

@riderx riderx marked this pull request as ready for review January 8, 2026 03:48
@riderx riderx merged commit 37b25cc into feature/domain-based-auto-join Jan 8, 2026
6 of 7 checks passed
@riderx riderx deleted the copilot/fix-action-step-error branch January 8, 2026 03:49
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.

3 participants