Skip to content

Loop#96

Open
javdl wants to merge 5 commits intomainfrom
loop
Open

Loop#96
javdl wants to merge 5 commits intomainfrom
loop

Conversation

@javdl
Copy link
Contributor

@javdl javdl commented Feb 26, 2026

No description provided.

javdl and others added 5 commits February 26, 2026 20:27
- Add fashionunitedTheme() plugin to astro.config.mjs (injects CSS, fonts, components)
- Replace 141-line tailwind.config.mjs with 15-line preset import
- Remove files now provided by plugin: button.tsx, card.tsx, constants.ts,
  Head.astro, PageTitle.astro, custom.css, globals.css
- Re-export cn() from plugin in src/lib/utils.ts for site-specific components
- Update register.astro imports to use plugin's Button/Card
- Update Layout.astro to import globals.css from plugin

Closes beads: developer-2y4, developer-3se, developer-9q2, developer-uik,
developer-2m0, developer-2g3, developer-1jy, developer-49h

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Closed developer-160 (about site migration) and developer-17h (button
regression). Blocked developer-2ly (npm publish) on missing npm auth.
Created developer-2ew for npm auth setup.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add .npmrc mapping @fashionunited scope to npm registry
- Update Dockerfile to copy .npmrc during install stages
- Restore epic filtering in PROMPT_build.md (review feedback)
- Plugin repo: added publishConfig, .npmrc, publish workflow

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…eveloper-2ly)

- Rename all imports from @fashionunited to @fuww scope
- Update .npmrc to use GitHub Packages registry
- Convert tailwind.config.mjs to use ESM imports (no more require())
- Package published as @fuww/starlight-plugin-theme@0.1.0

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5c433fc5cc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

"@fontsource-variable/inter": "^5.2.8",
"@fontsource-variable/lora": "^5.2.8",
"@fontsource/ibm-plex-mono": "^5.2.7",
"@fuww/starlight-plugin-theme": "github:fuww/starlight-plugin-theme#v0.1.0",

Choose a reason for hiding this comment

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

P1 Badge Regenerate Bun text lockfile after adding theme package

This adds @fuww/starlight-plugin-theme to dependencies, but the commit only updates bun.lockb and leaves bun.lock unchanged; meanwhile Dockerfile runs bun install --frozen-lockfile (lines 9 and 13), and bun install --help documents that this flag “Disallow[s] changes to lockfile.” In practice, CI/container builds will fail at install time because the package manifest now requires lockfile changes that frozen mode forbids, so bun.lock needs to be regenerated and committed with this dependency change.

Useful? React with 👍 / 👎.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@claude fix

Copy link

@claude claude bot Feb 27, 2026

Choose a reason for hiding this comment

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

Claude finished @javdl's task in 1m 27s —— View job


Fixing lockfile issue

  • Gather context — confirmed bun.lock (text lockfile) is missing @fuww/starlight-plugin-theme; Dockerfile uses --frozen-lockfile which requires both lockfiles to be in sync
  • Run bun install to regenerate bun.lock text lockfile
  • Commit and push the updated lockfile

Note: The bun install command requires approval in the current permission mode. Please approve it so I can regenerate bun.lock, or run it locally:

bun install
git add bun.lock bun.lockb
git commit -m "fix: regenerate bun.lock text lockfile for @fuww/starlight-plugin-theme dependency"
git push origin loop

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.

1 participant