Skip to content

Import and harden standalone lookinctl CLI - #2

Merged
nova286 merged 3 commits into
Developfrom
codex/upstream-client-cli
Jul 18, 2026
Merged

Import and harden standalone lookinctl CLI#2
nova286 merged 3 commits into
Developfrom
codex/upstream-client-cli

Conversation

@nova286

@nova286 nova286 commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • import the standalone lookinctl CLI from feat: 为 lookin 提供 lookincli 能力 hughkli/Lookin#63 while preserving the original author
  • expose scriptable device discovery, hierarchy JSON, screenshots, view-controller attribution, hit testing, semantic activation, and explicit debug-time mutations
  • pin CLI protocol sources and CocoaPods to the maintained LookinServer fork
  • build universal CLI artifacts alongside the desktop app in GitHub Actions
  • document upstream provenance and local adaptations

Distribution hardening

  • correct source and release URLs to the maintained fork
  • verify release archives with SHA-256 and code-signature checks before installation
  • remove automatic Gatekeeper quarantine deletion and npm install-time lifecycle scripts
  • package the npm distribution as GPL-3.0-only with the repository license
  • keep npm publishing explicitly opt-in

Validation

  • shell, Node.js, and workflow YAML syntax checks
  • universal arm64 x86_64 CLI build, ad-hoc signing, --version, and help smoke tests
  • no-Pods build from pinned LookinServer commit
  • npm tarball contents, metadata, license, and wrapper execution
  • installer checksum, signature, and installed-binary smoke test
  • universal macOS Release build of LookinClient
  • git diff --check origin/Develop...HEAD

The desktop build still emits existing dependency and deprecated-API warnings; no new build failure was introduced.

@nova286
nova286 marked this pull request as ready for review July 18, 2026 08:11
@nova286
nova286 merged commit 22251ec into Develop Jul 18, 2026
4 checks passed
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.

2 participants