Allow temporary IDs for all project operations

[Copilot]

• Fix TypeScript typecheck error in create_project.cjs (normalizeTemporaryId(temporaryId ?? ""))
• Verify all create_project_status_update tests pass (16/16)
• Verify formatting survives make fmt-cjs
• Build passes

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[mnkiefer]

@copilot fix js tests

[Copilot]

@copilot fix js tests

Fixed in 8cb353f. The TypeScript strict null check was failing because normalizeTemporaryId expects string but the JSDoc cast /** @type {string} */ (without parentheses) doesn't survive prettier's TypeScript parser — it gets stripped, leaving the type as string | null. Changed line 463 to normalizeTemporaryId(temporaryId ?? "") which satisfies TypeScript and is formatter-stable. The ?? "" fallback never triggers at runtime since the early return at line 350–356 already handles the null case.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://api.github.com/repos/github/gh-aw/contents/.github%2Fworkflows%2Faudit-workflows.md
  • Triggering command: /opt/hostedtoolcache/node/24.13.1/x64/bin/node /opt/hostedtoolcache/node/24.13.1/x64/bin/node --conditions node --conditions development --experimental-import-meta-resolve --require /home/REDACTED/work/gh-aw/gh-aw/actions/setup/js/node_modules/vitest/suppress-warnings.cjs /home/REDACTED/work/gh-aw/gh-aw/actions/setup/js/node_modules/vitest/dist/workers/forks.js m k/gh-aw/node_mod-- git push�� -u origin k/gh-aw/gh-aw/actions/node_modules/.bin/git it git _modules/.bin/git git (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

Pull request overview

Adds support for referencing newly-created GitHub Projects via temporary IDs (e.g. aw_... / #aw_...) so subsequent project operations can resolve the real project URL.

Changes:

• Store a temporary-id → project URL mapping when create_project succeeds.
• Resolve temporary project IDs from the shared temporaryIdMap in create_project_status_update.
• Add unit tests covering temporary project ID resolution and not-found behavior for status updates.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
actions/setup/js/create_project_status_update.test.cjs	Adds tests validating temporary project ID resolution (with and without #) and failure behavior.
actions/setup/js/create_project_status_update.cjs	Resolves output.project when it’s a temporary ID by looking up projectUrl in the shared temp-id map.
actions/setup/js/create_project.cjs	Stores the created project URL in the shared temp-id map and returns temporaryId in the result.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

handleCreateProjectStatusUpdate’s parameter order doesn’t match how handlers are invoked by safe_output_handler_manager (it calls handlers as (message, resolvedTemporaryIds, temporaryIdMap)). With the current signature (message, temporaryIdMap, resolvedTemporaryIds), temporaryIdMap will be a plain object in production and temporaryIdMap.get(...) will throw. Update the handler to accept (message, resolvedTemporaryIds = {}, temporaryIdMap = null) (or normalize args), and build a Map via temporaryIdMap instanceof Map ? temporaryIdMap : loadTemporaryIdMapFromResolved(resolvedTemporaryIds) before doing lookups.

[Copilot]

This new temporaryIdMap.set(...) assumes the second handler argument is a Map, but safe_output_handler_manager invokes handlers as (message, resolvedTemporaryIds, temporaryIdMap). That means temporaryIdMap will be a plain object at runtime and this will throw. Adjust handleCreateProject to accept the shared Map as the third parameter (and/or normalize args like other handlers), then write to that Map.

Suggested change

	temporaryIdMap.set(normalizedTempId, { projectUrl: projectInfo.projectUrl });
	if (temporaryIdMap instanceof Map) {
	temporaryIdMap.set(normalizedTempId, { projectUrl: projectInfo.projectUrl });
	} else if (temporaryIdMap && typeof temporaryIdMap === "object") {
	// Fallback for plain-object maps
	temporaryIdMap[normalizedTempId] = { projectUrl: projectInfo.projectUrl };
	}

[Copilot]

These tests invoke the handler as handler(message, temporaryIdMap), but the handler manager calls handlers as (message, resolvedTemporaryIds, temporaryIdMap). After aligning the handler signature with the manager, update these tests to pass the temporaryIdMap in the same position (e.g. handler(message, {}, temporaryIdMap)) so they exercise the production calling convention.