Skip to content

[WIP] Copilot Request - #54706

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-9919-1036865607-00c4b18f-655c-4532-ae80-ba17c40ccb70
Closed

pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-9919-1036865607-00c4b18f-655c-4532-ae80-ba17c40ccb70

Conversation

Copilot AI commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>[deep-report] Actionable repo-slug validation errors (repoutil + 3 pkg/cli call sites)</issue_title>
<issue_description>### Description

pkg/repoutil/repoutil.go:20's SplitRepoSlug returns fmt.Errorf("invalid repo format: %s", slug) with no expected format or example, contradicting the repo's own .github/skills/error-messages/SKILL.md style guide. The same bare pattern is duplicated in pkg/cli/preconditions.go:170, pkg/cli/fetch.go:95, and pkg/cli/outcome_eval.go:518 (each phrased slightly differently: "invalid repository format", "invalid repository slug", "invalid repo for root tracing"). Verified live at all 4 locations on main (commit f2e8122).

Fix

Update all 4 call sites to the skill's template — what's wrong, what's expected, an example — using consistent wording, e.g.:

fmt.Errorf("invalid repo format %q — expected 'owner/repo' format (for example: 'github/gh-aw')", slug)

Update/add tests asserting the new message text. Run make fmt. No behavior change, string-only.

Expected Impact

Users/agents hitting a malformed repo slug get an actionable fix path instead of a bare echo of their input, reducing debugging round-trips.

Suggested Agent

Copilot coding agent (mechanical string change + test update).

Estimated Effort

Quick (< 1 hour)

Data Source

DeepReport Intelligence Briefing 2026-08-21, mining discussion #54543 (Repository Quality Improvement Report - Error Message Actionability), Tasks 1-2.

Generated by 🔬 Deep Report · agent · 106.7 AIC · ⌖ 8.62 AIC · ⊞ 12.4K · ◷

  • expires on Aug 23, 2026, 10:30 AM UTC-08:00

Comments on the Issue (you are @copilot in this section)

Copilot AI requested a review from pelikhan August 22, 2026 03:30
@pelikhan pelikhan closed this Aug 22, 2026
@github-actions
github-actions Bot deleted the copilot/fix-9919-1036865607-00c4b18f-655c-4532-ae80-ba17c40ccb70 branch August 30, 2026 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deep-report] Actionable repo-slug validation errors (repoutil + 3 pkg/cli call sites)

2 participants