Teach DeepReport to publish a ranked function-complexity hotlist by Copilot · Pull Request #44667 · github/gh-aw

Copilot · 2026-07-10T06:32:40Z

DeepReport was surfacing the function-length backlog and Sentrux complexity signal as separate debt indicators, but not converting them into one actionable refactor queue. This change teaches the workflow to synthesize those inputs into a single prioritized top-10 hotlist for pkg/workflow and pkg/cli.

Structural-debt hotlist generation
- Adds an explicit DeepReport step for overlapping lint-monster + daily-sentrux findings.
- Refreshes the current largefunc / function-length backlog from make golint-custom.
- Pulls current Sentrux complexity data from sentrux check . --json.
- Ranks candidates by:
  1. Sentrux complexity
  2. recent churn
  3. execution-path centrality in workflow/CLI code
Issue shape for the consolidated backlog
- Requires one of the 7 DeepReport issues to become a single consolidated hotlist when the overlap is meaningful.
- Defines a ### Prioritized Hotlist (Top 10) table with:
  - rank
  - function
  - file
  - complexity evidence
  - churn evidence
  - suggested refactor seam
- Requires an explicit note when churn is only available at file granularity.
Workflow capability updates
- Imports shared Sentrux setup into DeepReport so the workflow can gather current structural metrics directly.
- Expands allowed network access to include GitHub for Sentrux installation.
- Makes the shallow-clone guidance concrete by specifying git fetch --unshallow origin before churn analysis.

### Prioritized Hotlist (Top 10)

| Rank | Function | File | Complexity evidence | Churn evidence | Refactor seam |
|------|----------|------|---------------------|----------------|---------------|
| 1 | compileWorkflow | pkg/workflow/... | sentrux hotspot + largefunc | 90d file churn | split parse / validate / emit |

Generated by 👨‍🍳 PR Sous Chef · 4.34 AIC · ⌖ 5.93 AIC · ⊞ 7.1K · ◷
_{Comment /souschef to run again}

Generated by 👨‍🍳 PR Sous Chef · 11.3 AIC · ⌖ 6.23 AIC · ⊞ 7.1K · ◷
_{Comment /souschef to run again}

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

github-actions · 2026-07-10T07:31:36Z

🤖 PR Triage

Field	Value
Category	feature
Risk	🟡 Medium
Score	48 / 100
Action	🔄 batch_review
Batch	`pr-batch:workflow-chore`

Score breakdown: impact 22 + urgency 12 + quality 14

Rationale: Workflow-only change (deep-report.md/.lock.yml) adding function-complexity hotlist synthesis. Medium risk due to workflow behavior change. Grouped with similar workflow-chore PRs for batch review.

Generated by 🔧 PR Triage Agent · 44.7 AIC · ⌖ 5.13 AIC · ⊞ 5.4K · ◷

Copilot

Pull request overview

This PR enhances the DeepReport workflow so it can synthesize long-function (largefunc / function-length) and Sentrux complexity signals into a single, ranked “Top 10” structural-debt hotlist for pkg/workflow and pkg/cli, and adjusts workflow wiring to support collecting that data during runs.

Changes:

Adds a DeepReport instruction step to refresh lint + Sentrux evidence, compute churn, and rank an intersection set into a top-10 refactor queue.
Updates DeepReport’s issue-shaping guidance to require a consolidated “### Prioritized Hotlist (Top 10)” issue when overlap is meaningful.
Imports shared Sentrux setup and expands workflow network allowlisting to permit GitHub access for tool installation.

Show a summary per file

File	Description
.github/workflows/deep-report.md	Adds structural-debt hotspot synthesis instructions, imports Sentrux shared setup, and extends network allowlist to enable GitHub access.
.github/workflows/deep-report.lock.yml	Regenerates the compiled workflow to include the Sentrux import, network allowlist expansion, and Sentrux installation step.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Files reviewed: 2/2 changed files
Comments generated: 3
Review effort level: Low

+
+1. Refresh the authoritative long-function backlog from the current tree with `make golint-custom`, then extract only `largefunc` / `function-length` findings in `pkg/workflow` and `pkg/cli`.
+2. Run `sentrux check . --json` and capture the highest-complexity functions in `pkg/workflow` and `pkg/cli`.
+3. Compute recent churn for the candidate files with `git log --since="90 days ago" --numstat -- pkg/workflow pkg/cli`, excluding generated files such as `*.lock.yml` and `actions-lock.json`.


+1. Refresh the authoritative long-function backlog from the current tree with `make golint-custom`, then extract only `largefunc` / `function-length` findings in `pkg/workflow` and `pkg/cli`.
+2. Run `sentrux check . --json` and capture the highest-complexity functions in `pkg/workflow` and `pkg/cli`.
+3. Compute recent churn for the candidate files with `git log --since="90 days ago" --numstat -- pkg/workflow pkg/cli`, excluding generated files such as `*.lock.yml` and `actions-lock.json`.
+4. If the checkout is shallow, run `git fetch --unshallow origin` and then fetch the default branch history you need before churn analysis so the ranking uses real history instead of a shallow sample.


+          SENTRUX_VERSION="0.5.7"
+          curl -fsSL -o /tmp/gh-aw/agent/sentrux "https://github.com/sentrux/sentrux/releases/download/v${SENTRUX_VERSION}/sentrux-linux-x86_64"
+          chmod +x /tmp/gh-aw/agent/sentrux
+          sudo mv /tmp/gh-aw/agent/sentrux /usr/local/bin/sentrux
+          sentrux --version


gh-aw-bot · 2026-07-10T08:22:29Z

pr-sous-chef
@copilot please run the pr-finisher skill, ensure the branch is up to date, address any remaining review feedback, and rerun checks.

Run: https://github.com/github/gh-aw/actions/runs/29079112957

Generated by 👨‍🍳 PR Sous Chef · 8.44 AIC · ⌖ 5.58 AIC · ⊞ 7.1K · ◷
_{Comment /souschef to run again}

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot · 2026-07-10T08:43:27Z

pr-sous-chef
@copilot please run the pr-finisher skill, ensure the branch is up to date, address any remaining review feedback, and rerun checks.
...

Addressed the remaining review feedback and merged main into the branch in a5a130b. Local make fmt, make lint, make test-unit, make test, make recompile, and make agent-report-progress all passed. CI on this head is still stale after the agent push, so a maintainer needs to re-trigger checks before merge.

github-actions · 2026-07-10T10:09:48Z

Hey @app/copilot-swe-agent 👋 — great work synthesizing the lint-monster + daily-sentrux overlap into a single prioritized refactor hotlist for pkg/workflow and pkg/cli! The structured hotlist table shape and concrete ranking criteria (complexity → churn → centrality) make this immediately actionable.

This PR looks well-scoped and ready for review:

✅ Clear description explaining the problem and the solution
✅ Focused on a single concern (DeepReport hotlist generation)
✅ Tests updated — golden files and docker_firewall_pin_compile_test.go reflect the changes
✅ No new external dependencies introduced

Looks good to go! 🚀

Generated by ✅ Contribution Check · 115.9 AIC · ⌖ 13.8 AIC · ⊞ 6.2K · ◷

gh-aw-bot · 2026-07-10T13:26:38Z

@copilot please run the pr-finisher skill, address any remaining review feedback, refresh the branch if needed, and rerun checks.

Generated by 👨‍🍳 PR Sous Chef · 4.34 AIC · ⌖ 5.93 AIC · ⊞ 7.1K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-10T13:41:25Z

@copilot please run the pr-finisher skill, address any remaining review feedback, refresh the branch if needed, and rerun checks.

...

No additional code changes were needed. The branch is already up to date with main at fc37683, the earlier review feedback is already present in the current workflow sources/lockfile, and I re-ran local make fmt, make lint, make test-unit, make test, make recompile, and make agent-report-progress successfully on this head. The PR checks on the agent-pushed head are still stale / action-required from the cloud-agent workflow model, so a maintainer needs to re-trigger CI before merge.

gh-aw-bot · 2026-07-10T14:17:30Z

@copilot please run the pr-finisher skill, address unresolved review comments, refresh the branch if needed, and rerun checks once ready.
Unresolved review threads:

Copilot: Teach DeepReport to publish a ranked function-complexity hotlist #44667 (comment)
Copilot: Teach DeepReport to publish a ranked function-complexity hotlist #44667 (comment)
Copilot: Teach DeepReport to publish a ranked function-complexity hotlist #44667 (comment)

Generated by 👨‍🍳 PR Sous Chef · 11.3 AIC · ⌖ 6.23 AIC · ⊞ 7.1K · ◷
_{Comment /souschef to run again}

Initial plan

b32743f

Copilot AI assigned Copilot and pelikhan Jul 10, 2026

Copilot started work on behalf of pelikhan July 10, 2026 06:33 View session

Copilot AI linked an issue Jul 10, 2026 that may be closed by this pull request

[deep-report] Publish a prioritized top-10 complexity hotlist from function-length + sentrux data #44577

Closed

Copilot AI and others added 2 commits July 10, 2026 06:48

Add deep-report structural debt hotlist guidance

f04b569

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Clarify deep-report hotlist guidance

2e15e66

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Publish prioritized top-10 complexity hotlist from audits~~ Teach DeepReport to publish a ranked function-complexity hotlist Jul 10, 2026

Copilot finished work on behalf of pelikhan July 10, 2026 06:55

Copilot AI requested a review from pelikhan July 10, 2026 06:55

github-actions Bot added pr-action:batch_review pr-agent:copilot-swe-agent pr-batch:workflow-chore pr-priority:medium pr-risk:medium pr-type:feature labels Jul 10, 2026

github-actions Bot mentioned this pull request Jul 10, 2026

[PR Triage Report] PR Triage Report — 2026-07-10 [Run 29076495188] #44682

Closed

pelikhan approved these changes Jul 10, 2026

View reviewed changes

pelikhan marked this pull request as ready for review July 10, 2026 07:35

Copilot AI review requested due to automatic review settings July 10, 2026 07:35

Copilot started reviewing on behalf of pelikhan July 10, 2026 07:35 View session

Copilot AI reviewed Jul 10, 2026

View reviewed changes

github-actions Bot mentioned this pull request Jul 10, 2026

[pr-sous-chef] Run report — 5 nudged, 0 skipped #44688

Closed

Copilot started work on behalf of gh-aw-bot July 10, 2026 08:23 View session

Copilot AI and others added 2 commits July 10, 2026 08:27

Plan review feedback follow-up

655ccb1

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Merge main and fix deep-report follow-ups

a5a130b

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot finished work on behalf of gh-aw-bot July 10, 2026 08:44

Copilot AI requested a review from gh-aw-bot July 10, 2026 08:44

github-actions Bot mentioned this pull request Jul 10, 2026

[Contribution Check Report] Contribution Check — 2026-07-10 #44641

Closed

This was referenced Jul 10, 2026

[pr-sous-chef] Run report — 0 nudged, 6 skipped #44708

Closed

[pr-sous-chef] Run report — 0 nudged, 9 skipped #44713

Closed

Merge branch 'main' into copilot/deep-report-publish-hotlist

fc37683

github-actions Bot mentioned this pull request Jul 10, 2026

[pr-sous-chef] Run report — 7 nudged, 2 skipped #44729

Closed

Copilot started work on behalf of gh-aw-bot July 10, 2026 13:27 View session

Copilot finished work on behalf of gh-aw-bot July 10, 2026 13:41

github-actions Bot mentioned this pull request Jul 10, 2026

[pr-sous-chef] Run report — 9 nudged, 0 skipped #44742

Closed

Copilot started work on behalf of gh-aw-bot July 10, 2026 14:18 View session

pelikhan closed this Jul 10, 2026

Copilot stopped work on behalf of gh-aw-bot due to an error July 10, 2026 14:30
The session was cancelled by the user.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Teach DeepReport to publish a ranked function-complexity hotlist#44667

Teach DeepReport to publish a ranked function-complexity hotlist#44667
pelikhan with Copilot wants to merge 6 commits into
mainfrom
copilot/deep-report-publish-hotlist

Copilot AI commented Jul 10, 2026 •

edited by github-actions Bot

Loading

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

gh-aw-bot commented Jul 10, 2026

Uh oh!

Copilot AI commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

gh-aw-bot commented Jul 10, 2026

Uh oh!

Copilot AI commented Jul 10, 2026

Uh oh!

gh-aw-bot commented Jul 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

Copilot AI commented Jul 10, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jul 10, 2026

🤖 PR Triage

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Review details

Uh oh!

gh-aw-bot commented Jul 10, 2026

Uh oh!

Copilot AI commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

gh-aw-bot commented Jul 10, 2026

Uh oh!

Copilot AI commented Jul 10, 2026

Uh oh!

gh-aw-bot commented Jul 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Copilot AI commented Jul 10, 2026 •

edited by github-actions Bot

Loading