fix: resolve documentation inconsistencies and minor code issues#82
Merged
Conversation
- Remove phantom `archgate mcp` command from README - Fix plugin path in CLAUDE.md to `../plugins/claude-code` - Add ARCH-008, ARCH-009 to CLAUDE.md ADR list - Replace `bun run build` with `bun run build:check` in CLAUDE.md - Update .prototools versions in CLAUDE.md (Bun 1.3.9, npm 11.11.0) - Add all undocumented commands/subcommands to CLAUDE.md table - Fix command description mismatches to match source code - Add review-context and session-context to CLI docs (en + pt-br) - Replace direct console.log with logInfo in git.ts and login-flow.ts - Update signup error message to direct users to `archgate login` - Add 3s timeout to WSL spawn in resolveCommand to prevent hangs - Add missing tests for adr/index.ts and plugin/index.ts
Deploying archgate-cli with
|
| Latest commit: |
cbe16fd
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://cc41b073.archgate-cli.pages.dev |
| Branch Preview URL: | https://fix-docs-inconsistencies.archgate-cli.pages.dev |
Extend ARCH-002 to enforce centralized logging in helper and engine files. The new `use-log-helpers` rule flags direct console.log/warn/info calls, requiring logInfo/logWarn/logDebug from helpers/log.ts instead. Command files are exempt (I/O layer), along with log.ts (canonical implementation), reporter.ts (check output system), and login-flow.ts (interactive device flow UI).
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
1 participant
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
archgate mcpcommand from README (no source/registration exists)archgate/claude-code-pluginto../plugins/claude-codebun run buildwithbun run build:checkin CLAUDE.mdplugin,plugin install,plugin url,login status,login logout,login refresh,session-context claude-code,session-context cursor.description()stringsreview-contextandsession-contextsections to CLI reference docs (en + pt-br for GEN-002 parity)console.logwithlogInfoingit.tsandlogin-flow.tsfor ARCH-003 consistencyarchgate logininstead of external URLresolveCommandto prevent test/runtime hangs on Windowsadr/index.tsandplugin/index.tscommand registrationTest plan
bun run validatepasses (lint, typecheck, format, 310 tests, ADR check, build check)adr/index.tsandplugin/index.tsverify subcommand registration