Skip to content

Temporarily disable figma/preview build steps until the package exists - #149

Merged
piaskowyk merged 2 commits into
mainfrom
copilot/fix-copilot-issue
Jul 31, 2026
Merged

Temporarily disable figma/preview build steps until the package exists#149
piaskowyk merged 2 commits into
mainfrom
copilot/fix-copilot-issue

Conversation

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

The deploy-docs CI job was failing because the workflow referenced figma/preview — a directory not yet present in the repo — causing an immediate crash when trying to npm ci there.

Changes

  • docs/package.json — Comment out the prebuild lifecycle script by renaming the key to "//prebuild" (standard JSON comment convention; npm ignores //-prefixed keys):
    "//prebuild": "node scripts/build-figma-preview.mjs"
  • .github/workflows/deploy-docs.yml — Comment out the "Install figma-preview dependencies" step with # prefixes; content is fully preserved.
  • docs/src/components/preview/embed.htmlPreview.astro has a static import embedHtml from './embed.html?raw' that hard-fails at Astro build time when the file is absent. Removed the gitignore entry and committed a placeholder so the build continues. This placeholder is replaced by the real generated file once figma/preview is wired back up.

Copilot AI changed the title [WIP] Fix issue with Copilot integration Temporarily disable figma/preview build steps until the package exists Jul 31, 2026
Copilot AI requested a review from piaskowyk July 31, 2026 15:44
@piaskowyk
piaskowyk marked this pull request as ready for review July 31, 2026 15:51
@piaskowyk
piaskowyk merged commit 4af374e into main Jul 31, 2026
1 check passed
@piaskowyk
piaskowyk deleted the copilot/fix-copilot-issue branch July 31, 2026 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants