Merge pull request gigsmart#122 from TheBushidoCollective/feat/github-releases

Add GitHub releases, tags, and artifacts

Author: jwaldrip

.ai-dlc/settings.yml

@@ -0,0 +1,14 @@
+git:
+  change_strategy: intent
+  default_branch: main
+  auto_merge: true
+  elaboration_review: true
+
+visual_review: false
+
+providers:
+  ticketing:
+    type: github-issues
+
+review_agents:
+  accessibility: true

.github/workflows/bump-plugin-version.yml

@@ -106,10 +106,12 @@ jobs:
           echo "new_version=$NEW_VERSION" >> $GITHUB_OUTPUT
 
       - name: Commit version bump
+        id: commit
         run: |
           # Check if there are any changes to commit
           if git diff --quiet; then
             echo "No version changes to commit"
+            echo "released=false" >> $GITHUB_OUTPUT
             exit 0
           fi
 
@@ -126,8 +128,55 @@ jobs:
             echo "Rebase conflict detected - another workflow likely pushed first"
             echo "Aborting rebase and exiting cleanly (next push will retry)"
             git rebase --abort 2>/dev/null || true
+            echo "released=false" >> $GITHUB_OUTPUT
             exit 0
           fi
 
           # Push the rebased commit
           git push origin main
+          echo "released=true" >> $GITHUB_OUTPUT
+
+      - name: Create and push git tag
+        if: steps.commit.outputs.released == 'true'
+        run: |
+          TAG="v${{ steps.bump-version.outputs.new_version }}"
+          git tag -f "$TAG"
+          git push -f origin "$TAG"
+
+      - name: Package plugin
+        if: steps.commit.outputs.released == 'true'
+        run: |
+          VERSION="${{ steps.bump-version.outputs.new_version }}"
+          zip -r "ai-dlc-plugin-${VERSION}.zip" plugin/ \
+            -x "plugin/node_modules/*" \
+            -x "plugin/*/node_modules/*" \
+            -x "plugin/*/.tsbuildinfo"
+
+      - name: Extract release notes
+        if: steps.commit.outputs.released == 'true'
+        run: |
+          VERSION="${{ steps.bump-version.outputs.new_version }}"
+          # Extract changelog section for this version
+          awk -v ver="$VERSION" '
+            /^## \[/ {
+              if (found) exit
+              if (index($0, "[" ver "]") > 0) { found=1; next }
+            }
+            found { print }
+          ' CHANGELOG.md > /tmp/release-notes.md
+          # Convert relative commit links to absolute URLs for release context
+          sed -i "s|../../commit/|https://github.com/${{ github.repository }}/commit/|g" /tmp/release-notes.md
+
+      - name: Create GitHub Release
+        if: steps.commit.outputs.released == 'true'
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: |
+          VERSION="${{ steps.bump-version.outputs.new_version }}"
+          TAG="v${VERSION}"
+          # Delete existing release on retry to ensure idempotency
+          gh release delete "$TAG" -y 2>/dev/null || true
+          gh release create "$TAG" \
+            --title "$TAG" \
+            --notes-file /tmp/release-notes.md \
+            "ai-dlc-plugin-${VERSION}.zip#AI-DLC Plugin (zip)"

bun.lock

@@ -4,7 +4,7 @@
       "command": "bash",
       "args": [
         "-c",
-        "command -v bun >/dev/null 2>&1 || { echo 'Error: Bun is required but not installed. Install it with: curl -fsSL https://bun.sh/install | bash' >&2; exit 1; }; exec bun run --cwd \"${CLAUDE_PLUGIN_ROOT}/mcp-server\" src/server.ts"
+        "cd \"${CLAUDE_PLUGIN_ROOT}/mcp-server\" && exec npx tsx src/server.ts"
       ]
     }
   }

plugin/.mcp.json

@@ -12,7 +12,8 @@
     "zod": "^3.23.0"
   },
   "devDependencies": {
-    "@types/bun": "^1.2.0",
+    "@types/node": "^22.0.0",
+    "tsx": "^4.19.0",
     "typescript": "^5.7.0"
   }
 }

plugin/mcp-server/package.json

@@ -1,11 +1,12 @@
-import { realpath } from "node:fs/promises"
+import { createServer, type Server as HttpServer } from "node:http"
+import { readFile, realpath } from "node:fs/promises"
 import { extname, join, resolve } from "node:path"
 import type { Server } from "@modelcontextprotocol/sdk/server/index.js"
 import { z } from "zod"
 import { getSession, updateQuestionSession, updateSession } from "./sessions.js"
 import type { QuestionAnswer } from "./sessions.js"
 
-let httpServer: ReturnType<typeof Bun.serve> | null = null
+let httpServer: HttpServer | null = null
 let actualPort: number | null = null
 
 /** Dependency-injected MCP server reference */
@@ -115,19 +116,16 @@ async function handleMockupGet(
 	}
 
 	try {
-		const file = Bun.file(resolved)
-		if (!(await file.exists())) {
-			return new Response("Not found", { status: 404 })
-		}
 		// Symlink-safe check: ensure resolved real path stays within base dir
 		const realResolved = await realpath(resolved).catch(() => null)
 		const realBase = await realpath(mockupsDir).catch(() => resolve(mockupsDir))
 		if (!realResolved || !realResolved.startsWith(realBase)) {
 			return new Response("Forbidden", { status: 403 })
 		}
+		const data = await readFile(resolved)
 		const ext = extname(resolved).toLowerCase()
 		const contentType = MIME_TYPES[ext] ?? "application/octet-stream"
-		return new Response(file, {
+		return new Response(data, {
 			headers: { "Content-Type": contentType },
 		})
 	} catch {
@@ -152,10 +150,6 @@ async function handleWireframeGet(
 	}
 
 	try {
-		const file = Bun.file(resolved)
-		if (!(await file.exists())) {
-			return new Response("Not found", { status: 404 })
-		}
 		// Symlink-safe check: ensure resolved real path stays within base dir
 		const realResolved = await realpath(resolved).catch(() => null)
 		const realBase = await realpath(session.intent_dir).catch(() =>
@@ -164,9 +158,10 @@ async function handleWireframeGet(
 		if (!realResolved || !realResolved.startsWith(realBase)) {
 			return new Response("Forbidden", { status: 403 })
 		}
+		const data = await readFile(resolved)
 		const ext = extname(resolved).toLowerCase()
 		const contentType = MIME_TYPES[ext] ?? "application/octet-stream"
-		return new Response(file, {
+		return new Response(data, {
 			headers: { "Content-Type": contentType },
 		})
 	} catch {
@@ -281,22 +276,18 @@ function handleRequest(req: Request): Response | Promise<Response> {
 	return new Response("Not Found", { status: 404 })
 }
 
-export function startHttpServer(): number {
+export async function startHttpServer(): Promise<number> {
 	if (httpServer && actualPort !== null) {
 		return actualPort
 	}
 
 	const basePort = Number.parseInt(process.env.AI_DLC_REVIEW_PORT ?? "8789", 10)
-	let port = basePort
 	const maxAttempts = 10
 
 	for (let i = 0; i < maxAttempts; i++) {
+		const port = basePort + i
 		try {
-			httpServer = Bun.serve({
-				port,
-				hostname: "127.0.0.1",
-				fetch: handleRequest,
-			})
+			await listenOnPort(port)
 			actualPort = port
 			console.error(`Review HTTP server listening on http://127.0.0.1:${port}`)
 			return port
@@ -306,7 +297,6 @@ export function startHttpServer(): number {
 				"code" in err &&
 				(err as NodeJS.ErrnoException).code === "EADDRINUSE"
 			) {
-				port++
 				continue
 			}
 			throw err
@@ -317,3 +307,61 @@ export function startHttpServer(): number {
 		`Could not find available port (tried ${basePort}-${basePort + maxAttempts - 1})`,
 	)
 }
+
+function listenOnPort(port: number): Promise<void> {
+	return new Promise((resolve, reject) => {
+		const server = createServer(async (req, res) => {
+			// Build a Web API Request from the incoming Node request
+			const url = `http://127.0.0.1:${port}${req.url ?? "/"}`
+			const headers = new Headers()
+			for (const [key, value] of Object.entries(req.headers)) {
+				if (value) {
+					if (Array.isArray(value)) {
+						for (const v of value) headers.append(key, v)
+					} else {
+						headers.set(key, value)
+					}
+				}
+			}
+
+			let body: ArrayBuffer | null = null
+			if (req.method !== "GET" && req.method !== "HEAD") {
+				const chunks: Buffer[] = []
+				for await (const chunk of req) {
+					chunks.push(typeof chunk === "string" ? Buffer.from(chunk) : chunk)
+				}
+				const buf = Buffer.concat(chunks)
+				body = buf.buffer.slice(buf.byteOffset, buf.byteOffset + buf.byteLength)
+			}
+
+			const webRequest = new Request(url, {
+				method: req.method ?? "GET",
+				headers,
+				body,
+			})
+
+			try {
+				const webResponse = await handleRequest(webRequest)
+				res.writeHead(
+					webResponse.status,
+					Object.fromEntries(webResponse.headers.entries()),
+				)
+				const responseBody = await webResponse.arrayBuffer()
+				res.end(Buffer.from(responseBody))
+			} catch (err) {
+				console.error("HTTP handler error:", err)
+				res.writeHead(500)
+				res.end("Internal Server Error")
+			}
+		})
+
+		server.once("error", (err) => {
+			reject(err)
+		})
+
+		server.listen(port, "127.0.0.1", () => {
+			httpServer = server
+			resolve()
+		})
+	})
+}

plugin/mcp-server/src/http.ts

@@ -1,3 +1,4 @@
+import { spawn } from "node:child_process"
 import { readdir } from "node:fs/promises"
 import { join, resolve } from "node:path"
 import {
@@ -314,7 +315,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 		})
 
 		// Start HTTP server (idempotent)
-		const port = startHttpServer()
+		const port = await startHttpServer()
 		const reviewUrl = `http://127.0.0.1:${port}/review/${session.session_id}`
 
 		// Open browser
@@ -323,7 +324,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 				process.platform === "darwin"
 					? ["open", reviewUrl]
 					: ["xdg-open", reviewUrl]
-			Bun.spawn(cmd, { stdio: ["ignore", "ignore", "ignore"] })
+			spawn(cmd[0], cmd.slice(1), { stdio: "ignore", detached: true }).unref()
 		} catch (err) {
 			console.error("Failed to open browser:", err)
 		}
@@ -418,7 +419,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 		})
 
 		// Start HTTP server (idempotent)
-		const port = startHttpServer()
+		const port = await startHttpServer()
 		const questionUrl = `http://127.0.0.1:${port}/question/${session.session_id}`
 
 		// Open browser
@@ -427,7 +428,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 				process.platform === "darwin"
 					? ["open", questionUrl]
 					: ["xdg-open", questionUrl]
-			Bun.spawn(cmd, { stdio: ["ignore", "ignore", "ignore"] })
+			spawn(cmd[0], cmd.slice(1), { stdio: "ignore", detached: true }).unref()
 		} catch (err) {
 			console.error("Failed to open browser:", err)
 		}

plugin/mcp-server/src/server.ts

@@ -21,12 +21,6 @@
 			"$ref": "#/definitions/providersConfig",
 			"description": "External system providers for spec, ticketing, design, and comms"
 		},
-		"workflow_mode": {
-			"type": "string",
-			"enum": ["interactive", "autonomous"],
-			"default": "interactive",
-			"description": "interactive: pause for user approval at key decision points. autonomous: proceed automatically, only pause on blockers."
-		},
 		"granularity": {
 			"type": "string",
 			"enum": ["coarse", "standard", "fine"],

plugin/schemas/settings.schema.json

@@ -39,8 +39,7 @@ Autopilot is designed for straightforward features. It pauses and returns contro
 1. **Pause on blockers or ambiguity** - Never guess. If elaboration or execution encounters something unclear, stop and ask.
 2. **Pause if elaboration generates more than 5 units** - Confirm scope with the user before proceeding to execution. More than 5 units suggests the feature may be too large or complex for autopilot.
 3. **Pause before creating PR** - Always confirm with the user before delivery. Show a summary of what was built across all units.
-4. **Config-aware mode** - Uses `workflow_mode: autonomous` from project config (`.ai-dlc/settings.yml`) if available. Falls back to interactive mode for any decision that could go wrong.
-5. **No silent failures** - If any phase fails, stop immediately and report what happened. Do not attempt to recover autonomously from phase-level failures.
+4. **No silent failures** - If any phase fails, stop immediately and report what happened. Do not attempt to recover autonomously from phase-level failures.
 
 ---