A section with no markers still runs its updater

[Jamie-BitFlight]

What happens

src/sections/index.ts guards each section with:

const [startToken, stopToken] = inputs.readmeEditor.getTokenIndexes(section);
if (startToken === -1 || stopToken === -1) {
  return {};
}

getTokenIndexes returns [] when a marker is missing, so both values are undefined and the guard never fires. Every configured section's updater runs whether or not the README has its markers.

Consequence

The README itself is safe, because updateSection finds no markers and writes nothing. The updater's other effects still happen:

• updateBranding writes the SVG file and logs branding svg successfully created for a README with no branding section.
• The action's sections output reports content for sections the README does not have.

Related

#644 asks for a warning when a section's markers are missing. This guard is where that check belongs, and today it never runs.

[Jamie-BitFlight]

Groomed

Reproduced on main: getTokenIndexes returns [] for a missing section, so the === -1 guard never fires. Ships in PR B with #644 and #641. PR B comes after PR A (#691), because the guard reads PR A's result (found, missing or ambiguous).

Acceptance criteria

• A section that is missing, has only half a pair, or is ambiguous does not run its updater.
• A test checks that a README with no branding markers writes no SVG and has no branding key in the sections output.

---

Generated by Claude Code

[Jamie-BitFlight]

Scope revised

The guard should not skip updaters. The sections output (documented in action.yml) reports every configured section, and that depends on every updater running. The README is already safe, because updateSection writes only where the markers are.

The real defect is the image: generateImgMarkup writes the SVG even when the README has no section that shows it. The fix removes the dead guard and writes the image only when the branding or title section is present. The groomed acceptance criterion "does not run its updater" is replaced by "writes no image without a section that shows it".

---

Generated by Claude Code

[Jamie-BitFlight]

🎉 This issue has been resolved in version 2.0.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀