Skip to content

Feat/view port option#186

Open
riley0227 wants to merge 6 commits intodevfrom
feat/view-port-option
Open

Feat/view port option#186
riley0227 wants to merge 6 commits intodevfrom
feat/view-port-option

Conversation

@riley0227
Copy link
Copy Markdown
Contributor

Summary

What does this PR change?

Type

  • Bug fix
  • Feature
  • Refactor
  • Docs
  • Test
  • CI

Changes

Validation

How did you test this?

Checklist

  • Tests pass
  • Smoke tests pass
  • No raw errors introduced
  • CLI output follows Ix format

TannerTorrey3 and others added 6 commits April 8, 2026 23:58
Added a Discord link to the README.
Removed Homebrew installation instructions from README.
Fixes 6 Dependabot alerts (4 high, 2 moderate) caused by vulnerable
Vite versions pulled in transitively via vitest.

- ix-cli: overrides vite to >=7.3.2 <8.0.0 (was 7.3.1 via vitest 3.2.4)
- core-ingestion: overrides vite to 8.0.5 (was 8.0.0 via vitest 4.1.0)
- core-ingestion: pins @emnapi/core and @emnapi/runtime as devDeps to
  ensure cross-platform lock file compatibility (Linux CI vs Windows dev)
- ci: upgrades Node.js from 22.14 to 24 to match lock file environment

Co-authored-by: Joseph Mikhail <jsmikhai@ucsd.edu>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add -p shorthand alias for --port
- Detect port conflicts before spawning the server
- Verify server readiness after spawn instead of assuming success
- Track port alongside PID in compass.json for accurate status display
Revert over-engineered STATE_FILE/readiness-signal refactor and keep
only the minimal port-related additions: -p/--port on the parent view
command, and a pre-flight isPortInUse check with a helpful error.
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.

4 participants