Skip to content

Remove phantom permission factory documentation - #54015

Merged
pelikhan merged 2 commits into
mainfrom
copilot/spec-librarian-fix-phantom-functions
Aug 19, 2026
Merged

Remove phantom permission factory documentation#54015
pelikhan merged 2 commits into
mainfrom
copilot/spec-librarian-fix-phantom-functions

Conversation

CopilotAI commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

The workflow package reference listed two permission factory constructors that do not exist in permissions_factory.go.

  • Permissions Factory reference
    • Removed NewPermissionsContentsReadIssuesWriteDiscussionsWrite()
    • Removed NewPermissionsContentsReadProjectsWrite()

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
CopilotAI changed the title [WIP] Fix phantom functions in permissions factory documentationRemove phantom permission factory documentationAug 19, 2026
CopilotAI requested a review from pelikhanAugust 19, 2026 13:41
@github-actions

Copy link
Copy Markdown
Contributor

PR Triage

Category: docs · Risk: low · Score: 20/100 (impact 5 + urgency 5 + quality 10)
Recommended action:defer

Tiny doc cleanup removing references to nonexistent functions; no urgency.

Automated triage — see run report for full details.

Generated by 🔧 PR Triage Agent · auto · 90.2 AIC · ⌖ 2.82 AIC · ⊞ 8.3K ·

@pelikhan
pelikhan marked this pull request as ready for review August 19, 2026 22:53
CopilotAI balanced review requested due to automatic review settings August 19, 2026 22:53
@pelikhan
pelikhan merged commit a9176a3 into mainAug 19, 2026
@pelikhan
pelikhan deleted the copilot/spec-librarian-fix-phantom-functions branch August 19, 2026 22:54

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes two nonexistent permission factory constructors from the workflow package reference, resolving #54014.

Changes:

  • Removed two phantom API entries from the permissions factory table.
  • Aligns documentation with permissions_factory.go.
Show a summary per file
FileDescription
pkg/workflow/README.mdRemoves references to nonexistent constructors.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.2

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[spec-librarian] Specification Audit - 2026-08-19 - 1 issue found

3 participants

@pelikhan