Polish public repository hygiene#21
Draft
AmeerJ97 wants to merge 2 commits into
Draft
Conversation
Task: TASK-01 Kind: maintenance Base: bbf4fef Evidence: git diff --check; bun run format; bun run lint; bun run typecheck; bun run test; bun run build Remote-Evidence: branch protection verified; topics/homepage/wiki updated; vulnerability alerts and private vulnerability reporting enabled
Task: FIX-01 Kind: fix Base: d78ed4c Evidence: git diff --check; bun run format; bunx --bun vitest run src/utils/install-manager.test.ts; bun run lint; bun run typecheck; bun run test; bun run build; bun run build:binary Local-Evidence: grok resolves to official x.ai CLI; grok-vertex resolves to ~/.local/bin/grok-vertex and reports 1.1.6
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
grok-vertexunder~/.grok-vertex/bin, whilegrokremains available for the official x.ai installer.Root cause / rationale
The repository is public and starting to receive automated dependency activity, but some public maintenance surfaces were incomplete or settings-only: no live security policy yet, no repo topics/homepage, wiki was enabled despite docs living in-repo, blank issues were allowed, and local stale branches needed an explicit audit record instead of replaying old work wholesale.
There is also a real command collision with the official x.ai CLI. The official installer at
https://x.ai/cli/install.showns~/.grok/bin/grok,~/.grok/bin/agent,~/.grok/config.toml, and shell PATH setup. This fork previously used the samegrokcommand and~/.grok/binscript install target, so the two installers could overwrite or shadow each other. New fork installs now usegrok-vertexand~/.grok-vertexfor script-managed binaries/metadata.Verification
git diff --checkbun run formatbun run lint(passes with existing warnings in untouched source files)bun run typecheckbun run testbun run buildbun run build:binarybunx --bun vitest run src/utils/install-manager.test.tsHOME:bash install.sh --binary dist/grok-standalone --no-modify-pathgrokresolves to official x.ai CLI;grok-vertexresolves to this fork and reports1.1.6ghRisk and rollout
Low runtime risk. The code change is limited to script install/update/uninstall metadata and package bin aliases. Existing legacy script-managed installs under
~/.grokremain readable for update/uninstall compatibility, but new installs use~/.grok-vertexto avoid clobbering the official x.ai CLI.Docs / config impact
grok-vertexFixes #
Checklist