Yocto recipe review

Skill Higangssh/yocto-agent-skills/skills/yocto-recipe-review

Review and improve Yocto/OpenEmbedded recipes, bbappends, bbclasses, package splits, dependencies, licensing, source fetching, patch handling, install logic, systemd/init integration, PACKAGECONFIG, and modern BitBake override syntax. Use for .bb, .bbappend, .bbclass, recipe diffs, recipe modernization, packaging errors, or metadata code review.From its SKILL.md

Install

npx -y skills add Higangssh/yocto-agent-skills --skill yocto-recipe-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

6 stars6 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

SKILL.md

2.2 KB, 488 tokens by cl100k_base, as published. Nobody here has run it

Yocto Recipe Review

Use this skill for .bb, .bbappend, and .bbclass review. Optimize for release-aware, reproducible, package-correct metadata.

Review Checklist

Inspect:

SUMMARY
DESCRIPTION
HOMEPAGE
LICENSE
LIC_FILES_CHKSUM
SRC_URI
SRCREV
S
inherit
DEPENDS
PACKAGECONFIG
do_configure
do_compile
do_install
FILES:${PN}
RDEPENDS:${PN}
SYSTEMD_SERVICE:${PN}

Findings to Prioritize

old override syntax in a modern layer without a release reason
missing or vague LICENSE and LIC_FILES_CHKSUM
${AUTOREV} in production metadata
missing DEPENDS for headers, libraries, generated code tools, or -native tools
runtime dependency added to DEPENDS instead of RDEPENDS:<package>
installed files not covered by FILES:*
installing outside ${D}
host path or host tool contamination
unnecessary INSANE_SKIP
patch entries with missing upstream status or fuzz
bbappend that is too version-specific or too broad for the intended target

Modern Syntax

Prefer:

FILES:${PN} += "${bindir}/tool"
RDEPENDS:${PN} += "bash"
PACKAGECONFIG:append = " feature"
do_install:append() {
    install -d ${D}${bindir}
}

Avoid _append, _prepend, _remove, and old package override syntax unless the target release requires it.

References

Read references/variables-core.md for variable semantics.
Read references/classes-core.md for class-specific review.
Read references/qa-errors.md for package QA risks.
Use references/official-doc-map.md to route official docs.

Output

Lead with code-review findings:

Findings:
- [severity] file:line issue and Yocto impact

Suggested metadata:
...

Validation:
- bitbake -e <recipe>
- bitbake -c patch <recipe>
- bitbake <recipe>
- bitbake <image>

What ships with it: 4 files

24.5 KB alongside SKILL.md