Skip to content

Decouple avatar asset from app workspace#110

Open
NWYLZW wants to merge 1 commit into
mainfrom
codex/avatar-independent-deploy
Open

Decouple avatar asset from app workspace#110
NWYLZW wants to merge 1 commit into
mainfrom
codex/avatar-independent-deploy

Conversation

@NWYLZW

@NWYLZW NWYLZW commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove assets/avatar from the app root pnpm workspace so app frozen install no longer validates the avatar asset site's package.json
  • document the new Avatar Pages boundary: the target repo installs/builds independently and aliases app-source for avatar source/CSS
  • update GitHub Actions and package agent notes to keep future deploy changes on the independent path

Related

Verification

  • pnpm install --frozen-lockfile
  • pnpm -C packages/avatar test
  • pnpm exec dprint check .github/workflows/deploy-avatar.yml pnpm-workspace.yaml .github/AGENTS.md .oo/rules/maintenance/README.md .oo/rules/release/avatar-github-pages.md packages/avatar/AGENTS.md packages/avatar/README.md

Experience Review

  • 已判断是否需要沉淀经验
  • 如需要,已运行 $post-task-experience-review
  • reviewer PASS 后才进入 merge

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Experience Review

Before merge, make sure the PR body contains this completed checklist:

## Experience Review
- [x] 已判断是否需要沉淀经验
- [x] 如需要,已运行 `$post-task-experience-review`
- [x] reviewer PASS 后才进入 merge

Checklist to confirm:

  • Decide whether this PR produced reusable project experience.
  • If experience should be captured, run $post-task-experience-review.
  • Merge only after the independent reviewer reports PASS.

@NWYLZW NWYLZW force-pushed the codex/avatar-independent-deploy branch from b915772 to 9b5a0df Compare July 4, 2026 11:58
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