Skip to content

feat: add integrate alias for install - #235

Merged
nicknisi merged 1 commit into
mainfrom
tars/add-integrate-alias
Aug 27, 2026
Merged

feat: add integrate alias for install#235
nicknisi merged 1 commit into
mainfrom
tars/add-integrate-alias

Conversation

@workos-tars

@workos-tars workos-tars Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add workos integrate as an alias for workos install
  • surface the alias in human help and resolve it in JSON help
  • keep installer telemetry and pre-command behavior canonicalized to install

Validation

  • bun run vitest run src/utils/help-json.spec.ts src/utils/command-telemetry.spec.ts src/bin-default-command.integration.spec.ts (54 tests)
  • bun run typecheck
  • bun run lint
  • bun run format:check
  • bun run build
  • compiled binary smoke: root help lists [aliases: integrate]; workos integrate --help shows installer options

Requested by nick.nisi@workos.com via TARS

@greptile-apps

greptile-apps Bot commented Aug 26, 2026

Copy link
Copy Markdown

Greptile Summary

Adds workos integrate as an alias for the existing installer while preserving canonical help and telemetry behavior.

  • Registers integrate alongside install with the same options and handler.
  • Resolves the alias to install in structured help and command-name canonicalization.
  • Excludes the alias from the same pre-command unclaimed-environment warning as install.
  • Adds coverage for human help, JSON help resolution, and telemetry canonicalization.

Confidence Score: 5/5

The PR appears safe to merge with the alias consistently routed and canonicalized across runtime dispatch, help, middleware, and telemetry.

Both command spellings share the same installer builder and handler, the preflight ordering remains intact, and the shared alias map keeps machine-readable help and telemetry aligned with the canonical install command.

Important Files Changed

Filename Overview
src/bin.ts Registers integrate as a yargs alias sharing the installer path and aligns raw-token middleware behavior.
src/lib/command-aliases.ts Canonicalizes integrate to install for structured help and telemetry consumers.
src/bin-default-command.integration.spec.ts Verifies that human root help exposes the new alias.
src/utils/command-telemetry.spec.ts Verifies canonical telemetry naming for the alias.
src/utils/help-json.spec.ts Verifies that structured help requests for the alias resolve to the installer schema.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[workos integrate] --> B[Yargs alias registration]
  B --> C[Install builder and options]
  C --> D[AuthKit preflight]
  D --> E[Credential resolution]
  E --> F[Installer handler]
  A --> G[Canonical alias map]
  G --> H[JSON help: install]
  G --> I[Telemetry name: install]
Loading

Reviews (1): Last reviewed commit: "feat: add integrate alias for install" | Re-trigger Greptile

@workos-tars

workos-tars Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Greptile reviewed the current head at 5/5 with no findings. All CI and security checks have passed.

No code changes were needed. PR #235 remains open and ready for human review.

@workos-tars

workos-tars Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

PR #235 is merged. Nick’s approval covered the current head, and all CI and security checks passed.

@nicknisi
nicknisi merged commit 1adbe84 into main Aug 27, 2026
5 checks passed
@nicknisi
nicknisi deleted the tars/add-integrate-alias branch August 27, 2026 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant