Installation feedback from copilot

[RWOverdijk]

👋 Copilot has feedback on us adding graphify.

In .opencode/skills/graphify/references/update.md:64:

The code-only update path never creates .graphify_extract.json: Step 3A writes only .graphify_ast.json, while the later merge immediately reads .graphify_extract.json at line 93 (and the normal cleanup removes that file). Every incremental update containing only source-code changes will therefore fail. Write the empty semantic result and run Step 3C before entering the incremental merge.

In .opencode/skills/graphify/SKILL.md:53:

This executes a user-supplied question by textual substitution into a shell command. The same pattern is used for paths, URLs, and generated answers in the referenced runbooks; quotes, $(), or backticks in otherwise valid developer input can break or alter the command. Pass these values through argument arrays or a safely serialized file rather than interpolating them into shell/Python snippets throughout the integration.

This issue also appears on line 88 of the same file.

The second point makes sense, it's how this whole project functions, but I did notice it failing several times. I also noticed the llm pasting the full user prompt as the question instead of extracting from it what is relevant.

As a last, but separate note: I also noticed that installation adds graphify to ./opencode/opencode.json with the wrong path. It doesn't have to be added at all, opencode picks up plugins automatically. ETA: found a related issue for this point.

ETA: Huge fan, graphify is the first thing i set up for all projects I work on 😄

[RWOverdijk]

I am not allowed to edit, for some reason. One more:

.opencode/skills/graphify/SKILL.md:429

The suggested --force recovery is not implemented anywhere in this skill: no usage/parser branch handles the flag, and Step 4 always calls the same guarded to_json. As a result, a full rebuild after deletions repeats this failure; incremental updates that intentionally remove nodes also reach this guard from references/update.md:168 and cannot complete. Add an explicit force/overwrite path that preserves the old graph for rollback and bypasses the shrink guard only when requested.