Re-land canvas-apps sync reminder hook, fixed for Claude Code

Re-applies the hook from microsoft#320, which was reverted in microsoft#327 because it broke
Claude Code. The hook logic (inject-sync-reminder.cs) is restored byte-for-byte;
only the registration wiring changed.

Two defects in microsoft#320, either one fatal:

1. `"hooks": "hooks/hooks.json"` in .claude-plugin/plugin.json. Claude Code's
   manifest schema rejects a string path, failing installation outright with
   `Validation errors: hooks: Invalid input`. Claude auto-discovers
   hooks/hooks.json, so the field is unnecessary — model-apps and power-pages
   both ship working hooks without it.

2. The `userPromptTransformed` key in hooks/hooks.json. Claude Code reads that
   file too and rejects unrecognized event keys. The resulting hook-load-failure
   silently discards the whole file, and also disables the plugin's
   canvas-authoring MCP server:
     [ERROR] Failed to load hooks for canvas-apps
     [DEBUG] Plugin not available for MCP - error type: hook-load-failed

Registration is now split by host. Claude Code reads only
.claude-plugin/plugin.json; Copilot CLI prefers .plugin/plugin.json and falls
back to .claude-plugin/ only when absent. So the Copilot-only event lives inline
in .plugin/plugin.json, where Claude Code never sees it:

  Claude Code | hooks/hooks.json (auto-discovered) | UserPromptSubmit
  Copilot CLI | .plugin/plugin.json (inline)       | userPromptTransformed

Two events are required because the hosts share no usable one: Copilot accepts
UserPromptSubmit as an alias for userPromptSubmitted, but that event has no
output processing, so the reminder would be generated and silently discarded.

Also re-applies microsoft#320's removal of the deprecated generate-canvas-app skill,
which the revert restored, and corrects the AGENTS.md claim that Claude Code
"ignores the Copilot-only event and continues loading UserPromptSubmit" — the
behaviour that caused this outage.

Verified on Claude Code 2.1.220 and Copilot CLI 1.0.76-1: plugin installs and
loads (status enabled, MCP intact) and the reminder reaches the model in both
hosts.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: rggammon

plugins/canvas-apps/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "canvas-apps",
-  "version": "2.2.1",
+  "version": "2.2.2",
   "description": "Build Power Apps Canvas Apps using the Canvas Authoring MCP server.",
   "author": {
     "name": "Microsoft",

plugins/canvas-apps/.plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "canvas-apps",
-  "version": "2.2.1",
+  "version": "2.2.2",
   "description": "Build Power Apps Canvas Apps using the Canvas Authoring MCP server.",
   "author": {
     "name": "Microsoft",
@@ -9,6 +9,19 @@
   "homepage": "https://github.com/microsoft/power-platform-skills/",
   "repository": "https://github.com/microsoft/power-platform-skills/",
   "license": "MIT",
+  "hooks": {
+    "userPromptTransformed": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "dotnet run --file \"${PLUGIN_ROOT}/hooks/inject-sync-reminder.cs\"",
+            "timeout": 30
+          }
+        ]
+      }
+    ]
+  },
   "keywords": [
     "canvas-apps",
     "power-apps",

plugins/canvas-apps/AGENTS.md

@@ -19,10 +19,14 @@ claude --plugin-dir /path/to/plugins/canvas-apps
 ## Architecture
 
 ```
-.plugin/plugin.json            ← Open Plugins metadata (name, version, keywords)
+.claude-plugin/plugin.json     ← Claude Code metadata (Claude Code reads only this manifest)
+.plugin/plugin.json            ← Open Plugins metadata + inline Copilot userPromptTransformed hook
 .mcp.json                      ← MCP server config (canvas-authoring, auto-registered)
 AGENTS.md                      ← Plugin guidance for AI agents (this file)
 CLAUDE.md                      ← Symlink → AGENTS.md
+hooks/
+  hooks.json                   ← Claude Code UserPromptSubmit registration (Copilot hook is NOT here)
+  inject-sync-reminder.cs      ← File-based .NET app that emits the sync reminder for both hosts
 references/
   TechnicalGuide.md            ← YAML syntax, control selection, layout strategies, Power Fx patterns
   DesignGuide.md               ← Aesthetic guidelines, anti-patterns, design process
@@ -38,8 +42,6 @@ skills/
     SKILL.md                   ← Registers the Canvas Authoring MCP server with Claude Code
   add-data-source/
     SKILL.md                   ← Guides user to add a data source or connector in Studio, then verifies
-  generate-canvas-app/
-    SKILL.md                   ← [DEPRECATED] Redirects to canvas-app
 ```
 
 ## Skills
@@ -75,6 +77,30 @@ The `canvas-authoring` MCP server exposes the following tools:
 | `list_data_sources` | Lists all available data sources in the current authoring session |
 | `sync_canvas` | Syncs the current coauthoring session state from the server to a local directory, writing all YAML files |
 
+## Hooks
+
+Both hosts inject a reminder to call `sync_canvas` before acting, so the agent never edits stale
+local `.pa.yaml` files. The reminder is conditional in wording — the agent skips the sync when no
+coauthoring session is active or the request is unrelated to a canvas app.
+
+| Host | Registered in | Event |
+|------|---------------|-------|
+| Claude Code | `hooks/hooks.json` (auto-discovered) | `UserPromptSubmit` |
+| Copilot CLI | `.plugin/plugin.json` (inline `hooks`) | `userPromptTransformed` |
+
+Both run `hooks/inject-sync-reminder.cs` via `dotnet run --file`, which branches on its stdin
+payload to emit the output shape each host expects.
+
+Keep Copilot-specific events out of `hooks/hooks.json`: Claude Code rejects unrecognized hook
+event names and will fail to load the plugin, including its `canvas-authoring` MCP server.
+`.claude-plugin/plugin.json` must not declare a `"hooks"` field either — Claude auto-discovers
+`hooks/hooks.json`, and a path string there blocks installation.
+
+Keep both manifests. Claude Code reads only `.claude-plugin/plugin.json`; Copilot reads
+`.plugin/plugin.json`. Deleting `.plugin/plugin.json` as a duplicate would make Copilot fall back
+to `.claude-plugin/`, which carries no Copilot event, and the reminder would stop reaching Copilot
+with no error.
+
 ## Prerequisites
 
 Before the MCP server will start, you need:

plugins/canvas-apps/hooks/hooks.json

@@ -0,0 +1,16 @@
+{
+  "description": "Canvas Apps lifecycle hooks",
+  "hooks": {
+    "UserPromptSubmit": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "dotnet run --file \"${CLAUDE_PLUGIN_ROOT}/hooks/inject-sync-reminder.cs\"",
+            "timeout": 30
+          }
+        ]
+      }
+    ]
+  }
+}

plugins/canvas-apps/hooks/inject-sync-reminder.cs

@@ -0,0 +1,63 @@
+#:property PublishAot=false
+
+using System.Text.Json;
+
+// Prompt hook for the canvas-apps plugin.
+//
+// The hook injects guidance instead of calling sync_canvas directly because it only
+// receives the session cwd, not the per-app working directory. A blanket sync could
+// overwrite the wrong directory or clobber local edits that have not been pushed.
+const string Reminder =
+    "Canvas Apps coauthoring reminder: " +
+    "Before acting on this request, if a Canvas Authoring session is " +
+    "active (i.e. `connect` has already been called for a canvas app), first call " +
+    "the `sync_canvas` MCP tool targeting the app's working directory to pull the " +
+    "current session state into the local .pa.yaml files. This avoids editing stale " +
+    "YAML. If no coauthoring session is active yet, or this request is unrelated to " +
+    "a canvas app, skip the sync.";
+
+try
+{
+    var rawInput = await Console.In.ReadToEndAsync();
+    string? transformedPrompt = null;
+
+    try
+    {
+        using var input = JsonDocument.Parse(rawInput);
+        if (input.RootElement.ValueKind == JsonValueKind.Object &&
+            input.RootElement.TryGetProperty("transformedPrompt", out var prompt) &&
+            prompt.ValueKind == JsonValueKind.String)
+        {
+            transformedPrompt = prompt.GetString();
+        }
+    }
+    catch (JsonException)
+    {
+        // Hook failures must not block the user's prompt. If parsing fails, emit the
+        // Claude Code additional-context shape (Copilot will ignore it).
+    }
+
+    using var writer = new Utf8JsonWriter(Console.OpenStandardOutput());
+    writer.WriteStartObject();
+
+    if (transformedPrompt is not null)
+    {
+        writer.WriteString(
+            "modifiedTransformedPrompt",
+            $"{transformedPrompt}\n\n{Reminder}");
+    }
+    else
+    {
+        writer.WriteStartObject("hookSpecificOutput");
+        writer.WriteString("hookEventName", "UserPromptSubmit");
+        writer.WriteString("additionalContext", Reminder);
+        writer.WriteEndObject();
+    }
+
+    writer.WriteEndObject();
+    writer.Flush();
+}
+catch
+{
+    // Hooks are advisory and must never prevent the host from processing a prompt.
+}