pi-revit 0.1.3 → 0.1.5

package/README.md

@@ -49,7 +49,7 @@ LLM provider, like any Pi session.
 Close Revit, then in PowerShell:
 
 ```powershell
-npx -y pi-revit
+npx.cmd -y pi-revit
 ```
 
 This installs the Pi package, builds and deploys the Revit bridge add-in, creates the

package/bin/pi-revit.js

@@ -1,95 +1,95 @@
-#!/usr/bin/env node
-const { spawnSync } = require("node:child_process");
-const fs = require("node:fs");
-const path = require("node:path");
-
-const root = path.resolve(__dirname, "..");
-const scriptsDir = path.join(root, "scripts");
-
-function usage() {
-	console.log(`pi-revit installer\n\nUsage:\n  npx -y pi-revit\n\nWhat it does on Windows:\n  1. Runs: pi install npm:pi-revit\n  2. Builds and deploys the Revit bridge add-in\n  3. Creates the Documents\\pi-revit workspace and global pi-revit command\n\nClose Revit before running. Revit 2025, 2026, or 2027 and the matching .NET SDK are required.`);
-}
-
-function fail(message) {
-	console.error(`\npi-revit install failed: ${message}`);
-	process.exit(1);
-}
-
-function commandExists(command) {
-	return spawnSync("where.exe", [command], { stdio: "ignore" }).status === 0;
-}
-
-function run(title, command, args) {
-	console.log(`\n==> ${title}`);
-	const result = spawnSync(command, args, { stdio: "inherit", shell: false });
-	if (result.error) fail(`${command} could not be started: ${result.error.message}`);
-	if (result.status !== 0) process.exit(result.status ?? 1);
-}
-
-function runCmd(title, commandLine) {
-	console.log(`\n==> ${title}`);
-	// On Windows, npm/pi are usually .cmd shims. Run them through cmd.exe so PATHEXT
-	// resolution works from npx's Node process just like it does in PowerShell/CMD.
-	const result = spawnSync("cmd.exe", ["/d", "/s", "/c", commandLine], { stdio: "inherit" });
-	if (result.error) fail(`${commandLine} could not be started: ${result.error.message}`);
-	if (result.status !== 0) process.exit(result.status ?? 1);
-}
-
-function runPowerShellScript(scriptName) {
-	const scriptPath = path.join(scriptsDir, scriptName);
-	if (!fs.existsSync(scriptPath)) fail(`missing script: ${scriptPath}`);
-	run(scriptName, "powershell.exe", ["-ExecutionPolicy", "Bypass", "-File", scriptPath]);
-}
-
-function revitIsRunning() {
-	const result = spawnSync(
-		"powershell.exe",
-		["-NoProfile", "-Command", "if (Get-Process Revit -ErrorAction SilentlyContinue) { exit 0 } else { exit 1 }"],
-		{ stdio: "ignore" },
-	);
-	return result.status === 0;
-}
-
-function waitForEnter() {
-	console.log("\nRevit appears to be running. Close Revit now, then press Enter to continue (or Ctrl+C to cancel).");
-	try {
-		fs.readSync(0, Buffer.alloc(1), 0, 1);
-	} catch {
-		// Non-interactive shell: fall through to the second running check.
-	}
-}
-
-if (process.argv.includes("--help") || process.argv.includes("-h")) {
-	usage();
-	process.exit(0);
-}
-
-if (process.platform !== "win32") {
-	fail("pi-revit's Revit bridge installer supports Windows only.");
-}
-
-if (!commandExists("pi")) {
-	fail("the 'pi' command was not found on PATH. Install Pi first: npm install -g --ignore-scripts @earendil-works/pi-coding-agent");
-}
-
-if (!commandExists("dotnet")) {
-	fail("the 'dotnet' command was not found on PATH. Install the .NET SDK required by your Revit version.");
-}
-
-if (revitIsRunning()) {
-	waitForEnter();
-	if (revitIsRunning()) fail("Revit is still running. Close Revit and run 'npx -y pi-revit' again.");
-}
-
-console.log("pi-revit full installer");
-console.log("This installs the Pi package, deploys the Revit add-in, and creates the workspace/global command.");
-
-runCmd("Install the Pi package from npm", "pi install npm:pi-revit");
-runPowerShellScript("deploy.ps1");
-runPowerShellScript("setup-workspace.ps1");
-
-console.log("\npi-revit installed.");
-console.log("Next steps:");
-console.log("  1. Start Revit and click Always Load if prompted.");
-console.log("  2. Open a project.");
-console.log("  3. Run: pi-revit");
+#!/usr/bin/env node
+const { spawnSync } = require("node:child_process");
+const fs = require("node:fs");
+const path = require("node:path");
+
+const root = path.resolve(__dirname, "..");
+const scriptsDir = path.join(root, "scripts");
+
+function usage() {
+	console.log(`pi-revit installer\n\nUsage:\n  npx.cmd -y pi-revit\n\nWhat it does on Windows:\n  1. Runs: pi install npm:pi-revit\n  2. Builds and deploys the Revit bridge add-in\n  3. Creates the Documents\\pi-revit workspace and global pi-revit command\n\nClose Revit before running. Revit 2025, 2026, or 2027 and the matching .NET SDK are required.`);
+}
+
+function fail(message) {
+	console.error(`\npi-revit install failed: ${message}`);
+	process.exit(1);
+}
+
+function commandExists(command) {
+	return spawnSync("where.exe", [command], { stdio: "ignore" }).status === 0;
+}
+
+function run(title, command, args) {
+	console.log(`\n==> ${title}`);
+	const result = spawnSync(command, args, { stdio: "inherit", shell: false });
+	if (result.error) fail(`${command} could not be started: ${result.error.message}`);
+	if (result.status !== 0) process.exit(result.status ?? 1);
+}
+
+function runCmd(title, commandLine) {
+	console.log(`\n==> ${title}`);
+	// On Windows, npm/pi are usually .cmd shims. Run them through cmd.exe so PATHEXT
+	// resolution works from npx's Node process just like it does in PowerShell/CMD.
+	const result = spawnSync("cmd.exe", ["/d", "/s", "/c", commandLine], { stdio: "inherit" });
+	if (result.error) fail(`${commandLine} could not be started: ${result.error.message}`);
+	if (result.status !== 0) process.exit(result.status ?? 1);
+}
+
+function runPowerShellScript(scriptName) {
+	const scriptPath = path.join(scriptsDir, scriptName);
+	if (!fs.existsSync(scriptPath)) fail(`missing script: ${scriptPath}`);
+	run(scriptName, "powershell.exe", ["-ExecutionPolicy", "Bypass", "-File", scriptPath]);
+}
+
+function revitIsRunning() {
+	const result = spawnSync(
+		"powershell.exe",
+		["-NoProfile", "-Command", "if (Get-Process Revit -ErrorAction SilentlyContinue) { exit 0 } else { exit 1 }"],
+		{ stdio: "ignore" },
+	);
+	return result.status === 0;
+}
+
+function waitForEnter() {
+	console.log("\nRevit appears to be running. Close Revit now, then press Enter to continue (or Ctrl+C to cancel).");
+	try {
+		fs.readSync(0, Buffer.alloc(1), 0, 1);
+	} catch {
+		// Non-interactive shell: fall through to the second running check.
+	}
+}
+
+if (process.argv.includes("--help") || process.argv.includes("-h")) {
+	usage();
+	process.exit(0);
+}
+
+if (process.platform !== "win32") {
+	fail("pi-revit's Revit bridge installer supports Windows only.");
+}
+
+if (!commandExists("pi")) {
+	fail("the 'pi' command was not found on PATH. Install Pi first: npm install -g --ignore-scripts @earendil-works/pi-coding-agent");
+}
+
+if (!commandExists("dotnet")) {
+	fail("the 'dotnet' command was not found on PATH. Install the .NET SDK required by your Revit version.");
+}
+
+if (revitIsRunning()) {
+	waitForEnter();
+	if (revitIsRunning()) fail("Revit is still running. Close Revit and run 'npx.cmd -y pi-revit' again.");
+}
+
+console.log("pi-revit full installer");
+console.log("This installs the Pi package, deploys the Revit add-in, and creates the workspace/global command.");
+
+runCmd("Install the Pi package from npm", "pi install npm:pi-revit");
+runPowerShellScript("deploy.ps1");
+runPowerShellScript("setup-workspace.ps1");
+
+console.log("\npi-revit installed.");
+console.log("Next steps:");
+console.log("  1. Start Revit and click Always Load if prompted.");
+console.log("  2. Open a project.");
+console.log("  3. Run: pi-revit");

package/package.json

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

package/scripts/setup-workspace.ps1

@@ -43,9 +43,15 @@ Write-Host "Workspace ready: $WorkspaceDir" -ForegroundColor Green
 $pi = Get-Command pi -ErrorAction SilentlyContinue
 if ($pi) {
     $binDir = Split-Path $pi.Source -Parent
-    $globalCmd = Get-Content (Join-Path $templates 'pi-revit-global.cmd')
-    $globalCmd = $globalCmd -replace '^set "WORKSPACE=.*$', ('set "WORKSPACE=' + $WorkspaceDir + '"')
-    Set-Content -Path (Join-Path $binDir 'pi-revit.cmd') -Value $globalCmd -Encoding Ascii
+    # Literal line swap (no regex): the workspace path may contain characters
+    # like $ that a -replace replacement string would misinterpret.
+    $globalCmd = Get-Content (Join-Path $templates 'pi-revit-global.cmd') | ForEach-Object {
+        if ($_ -like 'set "WORKSPACE=*') { 'set "WORKSPACE=' + $WorkspaceDir + '"' } else { $_ }
+    }
+    # cmd.exe parses .cmd files in the console's OEM code page: write the launcher
+    # in that encoding so workspace paths with non-ASCII characters (user names,
+    # localized folder names) stay intact.
+    Set-Content -Path (Join-Path $binDir 'pi-revit.cmd') -Value $globalCmd -Encoding Oem
     Write-Host "Global command installed: $(Join-Path $binDir 'pi-revit.cmd')" -ForegroundColor Green
     Write-Host 'Run pi-revit from any terminal (pi-revit <project> for a project folder).'
 } else {

package/scripts/uninstall.ps1

@@ -66,24 +66,30 @@ if ($pi) {
 # 3. Bridge runtime folder (connection token; recreated on each Revit start).
 Remove-IfExists (Join-Path $env:APPDATA 'RevitBridge') 'bridge runtime folder'
 
-# 4. Pi package registration. Mirror the documented install command exactly:
-#    `pi install ./` stores the package keyed by a path resolved relative to Pi's
-#    settings file, so removal must use the same `./` form from the repo root.
-#    An absolute path may not match the original registration. The repo root is
-#    the parent of this script's folder.
+# 4. Pi package registration. Pi keys a package by its install source: the npm
+#    installer (npx.cmd -y pi-revit) registers `npm:pi-revit`, while an install
+#    from a clone of this repo (`pi install ./`) registers the repo path resolved
+#    from the repo root. Try both forms; each succeeds only for the source that
+#    is actually registered, so a machine with both gets both removed.
 if ($pi) {
     $repoRoot = Split-Path $PSScriptRoot -Parent
     Push-Location $repoRoot
     try {
-        $global:LASTEXITCODE = 0
-        & pi remove ./
-        if ($LASTEXITCODE -eq 0) {
-            Write-Host 'Removed the pi-revit Pi package.' -ForegroundColor Green
+        $removed = @()
+        foreach ($source in @('npm:pi-revit', './')) {
+            $global:LASTEXITCODE = 0
+            & pi remove $source
+            if ($LASTEXITCODE -eq 0) {
+                $removed += $source
+            }
+        }
+        if ($removed.Count -gt 0) {
+            Write-Host "Removed the pi-revit Pi package ($($removed -join ', '))." -ForegroundColor Green
         } else {
-            Write-Warning "pi remove exited with code $LASTEXITCODE; run 'pi remove ./' from this repo manually."
+            Write-Warning "No pi-revit Pi package registration was found. If pi still lists it, run 'pi remove npm:pi-revit' (npm install) or 'pi remove ./' from this repo (install from source)."
         }
     } catch {
-        Write-Warning "Could not auto-remove the Pi package; run 'pi remove ./' from this repo manually. ($_)"
+        Write-Warning "Could not auto-remove the Pi package; run 'pi remove npm:pi-revit' (npm install) or 'pi remove ./' from this repo (install from source). ($_)"
     } finally {
         Pop-Location
     }