fix(docs): correct runbook Client ID wording and rewrap dependabot-repair header comment - #970
Merged
drmoisan merged 6 commits intoOct 2, 2026
Conversation
…x with a preflight-cleared minimal-audit plan Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ent rewrap Phase 1 edits with verification evidence; AC1 to AC4 checked off. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…locking) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
1 of 5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Suggested title
fix(docs): correct the runbook sources line to Client ID and rewrap the dependabot-repair header comment
Summary
Client ID location (steps 10–12)in place ofApp ID location (steps 10–12), matching the credential the runbook steps 10 and 22 and the workflow YAML sample use..github/workflows/dependabot-repair.ymlinto five comment lines of at most 100 characters each. The wording is unchanged and no YAML key, value or non-comment line changes.Why
The runbook sources line still described the credential as the App ID after the runbook and workflow moved to the Client ID, and one comment line in the workflow header was 145 characters wide. Both were recorded as residual wording defects in issue 952.
What Changed
docs/features/active/2026-09-19-dependabot-fanout-and-ci-failing-nuget-upgrades-911/runbooks/github-app-installation-token.runbook.md): line 301,App ID locationbecomesClient ID location. Line 102, which legitimately refers to the numeric App ID, is unchanged..github/workflows/dependabot-repair.yml): header comment lines 13 to 16 become five comment lines (lines 13 to 17). The file grows from 173 to 174 lines.docs/features/active/2026-09-30-dependabot-repair-runbook-and-workflow-comment-wording-952/: plan, evidence artifacts, and policy, code and feature audit artifacts.Architecture / How It Fits Together
No wiring changes. The workflow remains triggered by
workflow_runon completion of the CI workflow and filtered todependabot/head branches.Verification
Completed (from the evidence artifacts in the feature folder):
App ID locationoccurs on 0 lines,Client ID location (steps 10on 1 line, and the only remainingApp IDline is line 102.origin/main: 1 line added, 1 removed.origin/main: 4 lines added, 3 removed, every changed line a comment line.actionlint1.7.7 over the repository workflows: exit 0, no output.ok: true. Pester pass and total counts are not recorded locally and are read from the CI Pester job on this head.Recommended:
actionlintjob. This run is the evidence for themodified-workflow-needs-green-runrule (acceptance criterion 5);dependabot-repair.ymldefines noworkflow_dispatchtrigger, so no run of that workflow can occur against this branch.Backward Compatibility / Migration Notes
None. Documentation and comment text only.
Risks and Mitigations
#-prefixed added and removed lines, andactionlintpasses.Review Guide
evidence/tree is mechanical output and can be skimmed.Follow-ups
DEPENDABOT_REPAIR_APP_IDstill containsAPP_IDalthough it stores the Client ID. Renaming it touches repository configuration and is out of scope here (reported by the code review as informational).GitHub Auto-close