Add Qoder IDE installer target with global and local options#1277
Open
liuwenji007 wants to merge 1 commit into
Open
Add Qoder IDE installer target with global and local options#1277liuwenji007 wants to merge 1 commit into
liuwenji007 wants to merge 1 commit into
Conversation
Supports --location=global (writes MCP entry to Qoder's shared mcp.json under ~/Library/Application Support/Qoder/SharedClientCache/) and --location=local (superset: additionally writes a project-scoped .qoder/rules/codegraph.md rule file with trigger: always_on frontmatter). Key design decisions: - Rule file at .qoder/rules/codegraph.md (higher precedence than AGENTS.md, filename codegraph fully owns — no marker fence needed) - Qoder-specific playbook inlined (not the minimal shared block) because Qoder's client consumes MCP initialize.instructions weakly - No --path argument: relies on MCP roots/list handshake instead of workspaceFolder substitution (older Qoder builds don't expand it) - macOS absolute command resolution via login shell (same as Antigravity — handles stripped PATH from Dock/launchd) - Self-heals legacy AGENTS.md CodeGraph fence on upgrade - Atomic writes via atomicWriteFileSync for crash safety - Local uninstall preserves shared MCP entry (cross-project safe) Includes 10 test cases covering install/uninstall/idempotency/ self-heal/printConfig/detect across both locations.
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.
This pull request adds first-class support for the Qoder agent to CodeGraph, enabling automatic detection and configuration via
codegraph install. The installer now manages both global and project-local integration for Qoder, handling several agent-specific behaviors, and includes comprehensive tests to ensure correct setup, idempotency, and clean uninstallation. Documentation has been updated to reflect Qoder support throughout.Qoder Agent Integration:
qoderTarget) and updated theTargetIdtype to include'qoder'. [1] [2] [3]mcp.jsonin the platform config directory, ensuring idempotency and leaving sibling servers intact..qoder/rules/codegraph.mdrule file with Qoder-specific guidance and frontmatter, handling legacy migration and uninstall semantics.--pathor${workspaceFolder}), idempotency, and proper uninstall behavior.Documentation Updates:
README.mdto include Qoder as a supported agent, reflecting its presence in badges, install instructions, agent lists, and feature descriptions. [1] [2] [3] [4] [5] [6] [7]CHANGELOG.mddescribing the Qoder integration, agent-specific behaviors, and migration/compatibility notes.Code Maintenance:
Key design decisions:
Includes 10 test cases covering install/uninstall/idempotency/ self-heal/printConfig/detect across both locations.