pi-revit 0.1.9 → 0.2.1

package/extensions/pi-revit/index.ts

@@ -3,6 +3,7 @@ import { Type, type TSchema } from "typebox";
 import { readFile } from "node:fs/promises";
 import os from "node:os";
 import path from "node:path";
+import { version as packageVersion } from "../../package.json";
 
 interface BridgeInfo {
 	baseUrl: string;
@@ -195,6 +196,17 @@ function registerBridgeTool(pi: ExtensionAPI, descriptor: BridgeToolDescriptor)
 	});
 }
 
+/** `pi update --extensions` refreshes this package but not the deployed Revit add-in,
+ * so a newer extension can silently talk to an older bridge. The add-in reports the
+ * package version it was built from (stamped by scripts/build.ps1); any difference
+ * means the update is incomplete. */
+function versionMismatch(addinVersion: unknown): string | null {
+	const addin = typeof addinVersion === "string" && addinVersion.length > 0 ? addinVersion : null;
+	if (addin === packageVersion) return null;
+	const state = addin ? `still runs version ${addin}` : "predates version reporting";
+	return `pi-revit ${packageVersion} is installed, but the Revit add-in ${state} — the update is incomplete. Close Revit and run: npx.cmd -y pi-revit (or ask the agent to run scripts\\deploy.ps1 from the installed package, then restart Revit).`;
+}
+
 function registerPing(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "ping",
@@ -206,8 +218,9 @@ function registerPing(pi: ExtensionAPI) {
 		executionMode: "sequential",
 		async execute(_toolCallId, _params, signal) {
 			const payload = await bridgeRequest("/ping", { method: "GET" }, signal, 10_000);
+			const warning = versionMismatch((payload as { addinVersion?: string }).addinVersion);
 			return {
-				content: [{ type: "text", text: JSON.stringify(payload) }],
+				content: [{ type: "text", text: JSON.stringify(payload) + (warning ? `\nWARNING: ${warning}` : "") }],
 				details: payload,
 			};
 		},
@@ -219,6 +232,19 @@ export default async function revitConnector(pi: ExtensionAPI) {
 	// bridge is down, so it is never part of /tools discovery.
 	registerPing(pi);
 
+	// Surface an incomplete update (see versionMismatch) once per session, right
+	// where the user lands after running `pi update --extensions`. Bridge down at
+	// session start is the normal Revit-closed case: stay quiet.
+	pi.on("session_start", async (_event, ctx) => {
+		try {
+			const payload = (await bridgeRequest("/ping", { method: "GET" }, undefined, 3_000)) as { addinVersion?: string };
+			const warning = versionMismatch(payload.addinVersion);
+			if (warning) ctx.ui.notify(warning, "warning");
+		} catch {
+			// No bridge, no verdict.
+		}
+	});
+
 	let descriptors: BridgeToolDescriptor[];
 	try {
 		const payload = (await bridgeRequest("/tools", { method: "GET" }, undefined, DISCOVERY_TIMEOUT_MS)) as {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-revit",
-  "version": "0.1.9",
+  "version": "0.2.1",
   "description": "Native Pi connector for Autodesk Revit. Run npx.cmd -y pi-revit for the full Windows install.",
   "author": "Ahmad Altahlawi",
   "license": "MIT",

package/scripts/build.ps1

@@ -20,6 +20,17 @@ $ErrorActionPreference = 'Stop'
 $project = Join-Path $PSScriptRoot '..\src\Revit\RevitBridge.csproj'
 
 $buildArgs = @('build', $project, '-c', $Configuration)
+
+# Stamp the package version into the assembly so the bridge can report which release
+# the deployed add-in came from; the pi extension compares it against its own package
+# version at session start to detect partial updates.
+$packageJson = Join-Path $PSScriptRoot '..\package.json'
+if (Test-Path $packageJson) {
+    $packageVersion = (Get-Content $packageJson -Raw | ConvertFrom-Json).version
+    if ($packageVersion) {
+        $buildArgs += "-p:Version=$packageVersion"
+    }
+}
 if ($TargetFramework) {
     # Override TargetFrameworks (plural) rather than TargetFramework: NuGet restore ignores a
     # single -p:TargetFramework and still restores every framework the project declares, so a

package/skills/pi-revit/SKILL.md

@@ -30,7 +30,8 @@ Workflow guidance:
 - `get_elements` is the listing/counting primitive (`count_only: true` for bare counts). It returns identity fields only (id, name, category, typeName, levelId); read parameter values with `get_element_details`. Prefer a `category` or `of_class` scope when filtering by a parameter's display name.
 - The selection pipeline is `get_elements` -> ids -> `manage_selection` (action `set`); there is no inline filter on selection.
 - `set_parameters` is the home for bulk parameter writes AND renames (the `Name` parameter covers levels, views, sheets, types). One transaction per batch; per-element failures are reported.
-- Before writing `execute_csharp` code, verify unfamiliar classes/members with `search_api_docs` (works with no document open; first query builds the index and takes a few seconds).
+- Before writing `execute_csharp` code, verify unfamiliar classes/members with `search_api_docs` (works with no document open; first query builds the index and takes a few seconds). The top match carries its remarks, parameter docs, and returns inline, and every public API enum value is searchable — trust the result over guessing or web search; narrow the query to promote a different match into the top slot.
+- `export_documents` files its output under `Documents\pi-revit\Models\<model title>\exports` automatically when `output_dir` is omitted — keyed to the exported document, so it lands right even across many models. Pass `output_dir` only when the user names a different target.
 
 ## execute_csharp playbook
 

package/src/Revit/BridgeServer.cs

@@ -1,6 +1,7 @@
 using System.IO;
 using System.Net;
 using System.Net.Sockets;
+using System.Reflection;
 using System.Text;
 using System.Text.Json;
 
@@ -22,6 +23,15 @@ namespace RevitBridge
     {
         private const int DefaultPort = 47777;
         private const int MaxPort = 47797;
+
+        /// <summary>Package version stamped into the assembly by scripts/build.ps1. The pi
+        /// extension compares this against its own package version to detect partial
+        /// updates (npm package updated, add-in not redeployed); a build made without
+        /// the stamp reports its default assembly version instead.</summary>
+        private static readonly string AddinVersion =
+            typeof(BridgeServer).Assembly.GetCustomAttribute<AssemblyInformationalVersionAttribute>()?.InformationalVersion?.Split('+')[0]
+            ?? typeof(BridgeServer).Assembly.GetName().Version?.ToString(3)
+            ?? "unknown";
         private const int MaxRequestBytes = 4_000_000;
         private const int MaxContentChars = 12_000;
 
@@ -260,7 +270,7 @@ namespace RevitBridge
         private async Task<(int Status, object Response)> RouteAsync(string method, string path, Dictionary<string, string> query, string body, CancellationToken token)
         {
             if (method == "GET" && path == "/ping")
-                return (200, new { ok = true, service = "revit-bridge", revitVersion = _revitVersion, pid = Environment.ProcessId });
+                return (200, new { ok = true, service = "revit-bridge", revitVersion = _revitVersion, pid = Environment.ProcessId, addinVersion = AddinVersion });
 
             if (method == "GET" && path == "/tools")
             {