Skip to content

fix(website): use the OpenSpec pixel-O favicon on the docs site - #2142

Merged
alfred-openspec merged 1 commit into
mainfrom
claude/favicon-fumadocs-bdbe0c
Oct 24, 2026
Merged

fix(website): use the OpenSpec pixel-O favicon on the docs site#2142
alfred-openspec merged 1 commit into
mainfrom
claude/favicon-fumadocs-bdbe0c

Conversation

@TabishB

@TabishB TabishB commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replaces the placeholder indigo website/app/icon.svg with the pixel-O favicon set from openspec-landing/public/.
  • favicon-32.png → app/icon1.png, favicon-512.png → app/icon2.png, apple-touch-icon.png → app/apple-icon.png.
  • Next.js file conventions pick them up, so no metadata code changes.

Test plan

  • pnpm build in website/ passes
  • out/index.html emits rel="icon" links for 32x32 and 512x512 plus rel="apple-touch-icon" at 180x180

🤖 Generated with Claude Code

@TabishB
fix(website): use the OpenSpec pixel-O favicon on the docs site
c9fd27d 
Replace the placeholder indigo app/icon.svg with the favicon set from
openspec-landing (32px, 512px, and 180px apple-touch icon) so the docs
site matches the brand mark used elsewhere.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 24, 2026 •
edited
Loading

Copy link
Copy Markdown

Deploying openspec-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: c9fd27d
Status: ✅  Deploy successful!
Preview URL: https://0675fdda.openspec-docs.pages.dev
Branch Preview URL: https://claude-favicon-fumadocs-bdbe.openspec-docs.pages.dev

View logs

@TabishB
TabishB requested a review from a team as a code owner October 24, 2026 03:11
@TabishB
TabishB requested review from clay-good and removed request for a team October 24, 2026 03:11
@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters: 4
  • website/app/apple-icon.png is excluded by !**/*.png
  • website/app/icon.svg is excluded by !**/*.svg
  • website/app/icon1.png is excluded by !**/*.png
  • website/app/icon2.png is excluded by !**/*.png

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@openspec-cloud

openspec-cloud Bot commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

No PR-relevant drift confirmed.

AI-generated · A citation proves the line exists, not that it makes the case — verify before acting.
Checked the 0 requirements selected for this PR at c9fd27d (256 total).
This is not a full-repository clean result; see the check for coverage and any broader findings.
View results · Click Refresh, then Scan again in the check. Or comment /openspec-cloud.

alfred-openspec
alfred-openspec approved these changes Oct 24, 2026
View reviewed changes

@alfred-openspec alfred-openspec left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified against the canonical openspec-landing assets and the deployed preview. The docs site now uses the established pixel-O mark at the expected favicon and Apple touch sizes. This is the smallest viable fix for brand consistency, all required checks pass, and there is no docs-lab content impact.

@alfred-openspec
alfred-openspec added this pull request to the merge queue Oct 24, 2026
Merged via the queue into main with commit 6d04a1b Oct 24, 2026
31 checks passed
@alfred-openspec
alfred-openspec deleted the claude/favicon-fumadocs-bdbe0c branch October 24, 2026 03:59
SanHsien added a commit to SanHsien/OpenSpec that referenced this pull request Oct 25, 2026
@SanHsien @claude
fix: sync upstream 1.15 review (vitest/vite security, validate/parser…
7d38a76 
…/template fixes)

- vitest/@vitest/ui ^4.1.12, vite ^7.3.7 direct devDependency, launch-editor override (clears Dependabot Fission-AI#9-Fission-AI#12)
- pin refreshed GitHub Actions SHAs (setup-node 7.1.0, upload-artifact 7.0.2, create-github-app-token)
- port upstream approaches: zero-width TTY spinner (Fission-AI#2108), new-change schema line (Fission-AI#2120),
  explore capture table (Fission-AI#2086), glob artifact path bounding (Fission-AI#2104), backtick-info fence (Fission-AI#2131),
  unspaced ###Requirement (Fission-AI#2133), unparseable config.yaml (Fission-AI#2127), closing-run headings (Fission-AI#2145)
- triage upstream commits through 6d04a1b, PRs Fission-AI#2142, issues Fission-AI#2147; decisions in docs/DECISIONS.md

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@TabishB TabishB mentioned this pull request Oct 27, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alfred-openspec alfred-openspec alfred-openspec approved these changes
@clay-good clay-good Awaiting requested review from clay-good clay-good is a code owner automatically assigned from Fission-AI/openspec-maintainers

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TabishB @alfred-openspec