Skip to content

fix: CI tests, platform VSIX, auto-open threshold, new file formats#11

Closed
Rhyru9 wants to merge 3 commits into
mainfrom
dev
Closed

fix: CI tests, platform VSIX, auto-open threshold, new file formats#11
Rhyru9 wants to merge 3 commits into
mainfrom
dev

Conversation

@Rhyru9

@Rhyru9 Rhyru9 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

CI:

  • Added cargo test to the CI workflow, ensuring all 140 tests run on every push and pull request
  • Added restore-keys support to the release cache for improved cache hit rates and faster builds

Release:

  • Added platform-specific VSIX packages:

    • linux-x64
    • darwin-arm64
    • darwin-x64
    • win32-x64
  • Each VSIX now bundles the correct native binary for its target platform

VS Code extension:

  • Moved file size validation into CustomReadonlyEditorProvider.resolveCustomEditor

  • Added a 25 MB auto-open threshold to prevent large files from opening unintentionally

  • Set editor priority: default so supported file types automatically open with Glance

  • Improved binary detection:

    • glance.exe on Windows
    • glance on Linux and macOS
  • Added support for additional file formats:

    • .sql
    • .txt
    • .xml
    • .out
    • .ldjson
  • Removed the onDidOpenTextDocument redirect mechanism due to inconsistent behavior across platforms and editor workflows

Rhyru9 added 3 commits June 1, 2026 12:12
…stomReadonlyEditorProvider: size check in resolveCustomEditor

- Files >= 25MB auto-open in Glance, smaller files use text editor
- priority: default — intercepts supported file types
- Binary detection: glance.exe on Windows, glance on Linux/macOS
- New formats: .sql, .txt, .xml, .out, .ldjson
- Remove unreliable onDidOpenTextDocument redirect approach
  ci.yml:
  - Add cargo test (140 tests now run on every push/PR)

  release.yml:
  - Platform-specific VSIX: linux-x64, darwin-arm64, darwin-x64, win32-x64
  - Each VSIX bundles the correct binary for its OS
  - Add restore-keys to release cache
@Rhyru9 Rhyru9 closed this Jun 1, 2026
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