Skip to content

agent-graph-workflow-v7 - #743

Closed
robertmonka wants to merge 3 commits into
Graphify-Labs:v7from
robertmonka:agent-graph-workflow-v7
Closed

agent-graph-workflow-v7#743
robertmonka wants to merge 3 commits into
Graphify-Labs:v7from
robertmonka:agent-graph-workflow-v7

Conversation

@robertmonka

@robertmonka robertmonka commented May 5, 2026

Copy link
Copy Markdown

🧢 Changes

  • Splits assistant integration commands into clearer user-level and project-level flows:
    • graphify skill <platform> installs or refreshes the assistant skill.
    • graphify setup <platform> writes project-local assistant instructions and hooks.
  • Keeps deprecated install aliases working for compatibility, including graphify install --platform <platform> and platform-first graphify <platform> install forms.
  • Unifies platform skill configuration, source file resolution, install paths, remove behavior, and package-data coverage across supported assistants.
  • Adds .env loading for CLI and direct LLM usage, including project/home .env discovery, WSL Windows-home fallback, and Windows user environment support for common graphify/API variables.
  • Adds a Codex Kimi fast path: when MOONSHOT_API_KEY is set, semantic extraction can use graphify.llm.extract_corpus_parallel(..., backend="kimi") directly instead of dispatching Codex subagents.
  • Moves temporary workflow state under graphify-out/ so skill runs avoid root-level .graphify_* scratch files.
  • Adds graph-first Claude setup with a UserPromptSubmit reminder and a PreToolUse guard that blocks raw search/read/list tools until the graph has been used in the session.
  • Fixes extracted ID remapping so file-derived and entity-derived IDs are normalized consistently before cross-file call target resolution.
  • Updates README command examples and platform tables to reflect the new skill and setup command model.

☕️ Reasoning

The previous install surface mixed two different responsibilities: installing reusable assistant skills and configuring the current project to use a graph. This PR separates those flows while preserving old commands as aliases, which makes onboarding clearer without breaking existing users.

It also makes the graphify assistant workflow more reliable for larger or repeated runs. Kimi can be used directly when a Moonshot key is available, temporary files stay inside graphify-out/, Claude is nudged or guarded toward graph-first exploration, and .env loading makes API-key configuration work more consistently across local, Windows, and WSL environments.

🧪 Testing

  • Adds/updates tests for CLI command parsing, deprecated aliases, skill installation paths, setup/remove behavior, package-data coverage, and no-side-effect help commands.
  • Adds tests for .env loading precedence, quoted/exported values, process-env protection, and WSL Windows-home discovery.
  • Adds tests for Claude graph guard behavior before and after graph use, including Windows-style paths.
  • Adds skill contract tests for the Codex/Kimi fast path and shared Claude/Codex output locations.
  • Not run locally while updating this PR description.

@robertmonka
robertmonka force-pushed the agent-graph-workflow-v7 branch 2 times, most recently from 3f6dc3b to d12919b Compare May 7, 2026 09:43
@robertmonka
robertmonka force-pushed the agent-graph-workflow-v7 branch 9 times, most recently from ebb4c91 to b14ee33 Compare May 9, 2026 23:29
Extend func_node tracking to Swift, Kotlin, Go, C#, and PHP call
handlers so the AST node is consistently available. This fixes cases
where caller/callee relationships were missing node references needed
for downstream edge generation.

Normalize both file node IDs and entity stem-prefix IDs when remapping
absolute paths to project-relative paths. Entity IDs use
_file_stem(path) as their prefix, so remapping only whole-file IDs
left cross-file edges unresolved (Graphify-Labs#550).
Rename `graphify install` to `graphify skill <platform>` to clarify it
installs assistant skills rather than package binaries. Add vscode
platform support alongside existing claude, copilot, and others.

Load environment variables via `graphify.env.load_env()` on CLI startup
to support `GRAPHIFY_OUT` overrides and `CLAUDE_CONFIG_DIR` for custom
skill destinations.

Split generic `skill.md` into platform-specific files (`skill-claude.md`,
`skill-vscode.md`) so each assistant receives tailored instructions.
Update output path references from `.graphify_semantic_new.json` to
`graphify-out/.graphify_semantic_new.json` to match the configurable
output directory.

Refactor platform config to use helper functions for source and
destination paths, reducing duplication and enabling environment-based
overrides for Claude skills.
Refactor Codex integration from deprecated hooks.json to config.toml
with features.hooks flag, adding migration path for legacy setups.
Clean legacy hook entries during install to avoid duplicates.

Add Kimi Code to supported platforms list with skill directory
installation. Include tomli dependency for Python < 3.11 compatibility
to support tomllib fallback in tests and runtime.
@robertmonka
robertmonka force-pushed the agent-graph-workflow-v7 branch from b14ee33 to 51e7cc8 Compare May 10, 2026 06:53
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.

1 participant