Skip to content

Bug: dependabot-repair-runbook-and-workflow-comment-wording #952

Description

@drmoisan
  • Work Mode: minor-audit

Summary

The #929 review left two cosmetic defects in the Dependabot repair documentation and workflow. The runbook's sources line still cites "App ID location" although the procedure now records the Client ID, and the header comment in dependabot-repair.yml is about 150 characters long.

Environment

  • OS/version: n/a
  • Python version: n/a
  • Command/flags used: n/a
  • Data source or fixture: n/a

Steps to Reproduce

  1. Read docs/features/active/2026-09-19-dependabot-fanout-and-ci-failing-nuget-upgrades-911/runbooks/github-app-installation-token.runbook.md line 301.
  2. Read .github/workflows/dependabot-repair.yml line 14.

Expected Behavior

Line 301 reads "Client ID location", matching steps 10 and 22 and the YAML sample. The workflow header comment is wrapped to the repository's line length.

Actual Behavior

Line 301 reads "Private key generation and App ID location (steps 10-12)". The workflow header comment is about 150 characters long.

Logs / Screenshots

Impact / Severity

  • Blocker
  • High
  • Medium
  • Low

Source

From: docs/features/potential/2026-09-30-dependabot-repair-runbook-and-workflow-comment-wording.md

Activity

  1. drmoisan commented on Sep 30, 2026

    @drmoisan
    OwnerAuthor

    Restoring sections dropped by the promotion tooling.

    Suspected Cause / Notes

    The citation text drifted when #929 corrected the instructions. The instructions themselves (steps 10 and 22, and the YAML sample) are correct. Source: #929 code review (Minor) and policy audit NB-5.

    Proposed Fix / Validation Ideas

    • Reword runbook line 301 to "Client ID location", and wrap the dependabot-repair.yml line 14 header comment.
    • Confirm that actionlint still passes.
  2. drmoisan commented on Oct 2, 2026

    @drmoisan
    OwnerAuthor

    Delivered by PR 970 (#970), merged into main as ab14b12. All 5 acceptance criteria are checked off in the feature folder issue.md; CI run 36969197402 on head ff8c715 passed all 7 required checks including actionlint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions