Skip to content

Add Fengge V1 Codex pet#61

Open
qzl1-stack wants to merge 3 commits into
legeling:mainfrom
qzl1-stack:agent/submit-fengge-qzl1-stack
Open

Add Fengge V1 Codex pet#61
qzl1-stack wants to merge 3 commits into
legeling:mainfrom
qzl1-stack:agent/submit-fengge-qzl1-stack

Conversation

@qzl1-stack

@qzl1-stack qzl1-stack commented Jul 27, 2026

Copy link
Copy Markdown

Summary

Adds the V2 Codex pet 峰哥 as a focused three-file submission.

Duplicate research

  • Checked the current pets.json catalog (165 entries), canonical keys, names, works/collections, tags, and open issues.
  • No existing entry or issue matched 峰哥 / Fengge / original/qzl1-stack/fengge.

Authorship and source

  • Final pet asset author/adaptor: qzl1-stack.
  • The final spritesheet pixels were independently AI-generated from a private blessing video supplied as identity and motion reference.
  • The private reference video is not included in this pull request, and no source-video pixels were directly cropped or reused.
  • Public source_url is intentionally empty because the reference is private.

Usage terms

Personal and repository use only; non-commercial use only. Commercial use is prohibited.

Runtime and quality

  • Runtime: V2, spriteVersionNumber: 2
  • Atlas: 1536x2288, 8 columns × 11 rows
  • Standard actions, directional gait, 16 clockwise look directions, identity/prop consistency, baseline, continuity, and transparent edges were reviewed.
  • All four cardinal direction blind gates passed. Intermediate direction uncertainty was reviewed at labeled normal size with no wrong-quadrant pose or reversal.
  • Deterministic chroma despill and V2 atlas validation passed with no errors or warnings.

Validation

  • npm run validate:pr equivalent repository-script execution: passed
  • npm run lint equivalent Prettier check: passed
  • Isolated install test: passed
  • Final submission SHA-256: 0996BA4708BC7116500DF1BF16628E6331AA73B5A20CBAAE4961E219314A20BE

Contact sheet

The final contact sheet will be attached to this section through the GitHub editor; it is intentionally not committed to the pet directory.

Linked issue: none.

Summary by CodeRabbit

  • 新功能
    • 新增宠物元数据配置文件,补充宠物的标识、名称、描述及精灵图资源信息。
    • 新增宠物安装配置文件,支持关联宠物条目、精灵图资源映射,并补充作者与许可证等说明。

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@qzl1-stack, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 1 minute

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 25d67fce-332d-40be-a4e1-1e27f2e6420c

📥 Commits

Reviewing files that changed from the base of the PR and between 8cb568f and 0a9924d.

📒 Files selected for processing (1)
  • pets/fengge--qzl1-stack/pet.json
📝 Walkthrough

Walkthrough

新增“峰哥”宠物的运行时元数据和提交配置,包含展示信息、精灵图版本与路径、来源许可说明,以及 pet.jsonspritesheet.webp 的安装映射。

Changes

宠物元数据注册

Layer / File(s) Summary
宠物元数据与安装映射
pets/fengge--qzl1-stack/pet.json, pets/fengge--qzl1-stack/submission.json
新增宠物标识、展示名称、描述、精灵图配置、来源与许可证信息,并声明相关安装文件映射。

Estimated code review effort: 1 (Trivial) | ~3 minutes

Poem

兔子蹦跳看新家,
峰哥名字亮晶晶。
精灵图路径已铺好,
安装映射排成行。
元数据齐,我捧胡萝卜庆祝!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确概括了新增 Fengge Codex pet 的主要变更,与本次提交内容一致。
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@qzl1-stack qzl1-stack changed the title Add Fengge V2 Codex pet Add Fengge V1 Codex pet Jul 27, 2026

Copy link
Copy Markdown

Thanks for the focused three-file submission. I checked the current PR head locally and found one blocking version mismatch:

  • spritesheet.webp is 1536x2288 (8x11), which is the V2 atlas format.
  • pet.json currently sets "spriteVersionNumber": 1, so npm run validate:pr fails with:
fengge--qzl1-stack: v1 spritesheet.webp must be 1536x1872, got 1536x2288

Please change spriteVersionNumber back to 2 and update the PR title/summary so they consistently describe this as a V2 pet. The atlas SHA-256 I calculated is 0996ba4708bc7116500df1bf16628e6331aa73b5a20cbaae4961e219314a20be, matching the value stated in the PR description.

Please also attach the promised contact sheet (or let the repository preview workflow generate its artifact) and wait for the previews check to pass. Once the version mismatch is fixed and CI is green, this should be ready for another merge review.

@qzl1-stack

Copy link
Copy Markdown
Author

Thanks for the focused three-file submission. I checked the current PR head locally and found one blocking version mismatch:

  • spritesheet.webp is 1536x2288 (8x11), which is the V2 atlas format.
  • pet.json currently sets "spriteVersionNumber": 1, so npm run validate:pr fails with:
fengge--qzl1-stack: v1 spritesheet.webp must be 1536x1872, got 1536x2288

Please change spriteVersionNumber back to 2 and update the PR title/summary so they consistently describe this as a V2 pet. The atlas SHA-256 I calculated is 0996ba4708bc7116500df1bf16628e6331aa73b5a20cbaae4961e219314a20be, matching the value stated in the PR description.

Please also attach the promised contact sheet (or let the repository preview workflow generate its artifact) and wait for the previews check to pass. Once the version mismatch is fixed and CI is green, this should be ready for another merge review.

改好了

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