Skip to content

fix: correct hardcoded repo reference in bump-go.sh#8

Merged
Moshbbab merged 2 commits intotrunkfrom
copilot/fix-bump-go-script-repo-reference
Apr 6, 2026
Merged

fix: correct hardcoded repo reference in bump-go.sh#8
Moshbbab merged 2 commits intotrunkfrom
copilot/fix-bump-go-script-repo-reference

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 6, 2026

The bump-go.sh script had a hardcoded cli/cli repo reference causing the Bump Go workflow to fail when searching for existing PRs — it was querying the wrong repository entirely.

Change

  • .github/workflows/scripts/bump-go.sh line 93: --repo cli/cli--repo Moshbbab/cli
- existing_pr=$(gh search prs --repo cli/cli --match title "$PR_TITLE" --json title --jq "...")
+ existing_pr=$(gh search prs --repo Moshbbab/cli --match title "$PR_TITLE" --json title --jq "...")
Original prompt

Issue

The .github/workflows/scripts/bump-go.sh script contains an incorrect hardcoded repository reference on line 93.

Current (incorrect):

existing_pr=$(gh search prs --repo cli/cli --match title "$PR_TITLE" --json title --jq "map(select(.title == \"$PR_TITLE\") | .title) | length > 0")

Required (correct):

existing_pr=$(gh search prs --repo Moshbbab/cli --match title "$PR_TITLE" --json title --jq "map(select(.title == \"$PR_TITLE\") | .title) | length > 0")

Impact

The "Bump Go" workflow (.github/workflows/bump-go.yml) fails at runtime because the PR search references the wrong repository owner. The gh search prs command cannot find existing PRs in the correct repository.

Change Required

Replace cli/cli with Moshbbab/cli on line 93 of .github/workflows/scripts/bump-go.sh.

No other changes. No refactoring. Exact fix only.

The following is the prior conversation context from the user's chat exploration (may be truncated):

Assistant: Now let me check for GitHub Actions workflows and outdated forks more systematically:

Assistant: Now let me compile a comprehensive audit report based on the data I've gathered.

GitHub Repository Audit Report for @Moshbbab

Audit Date: 2026-04-06 | Total Repositories: 91


📊 Executive Summary

Your GitHub activity shows a highly active developer with significant focus on AI, real estate technology, and LLM applications. However, the audit identified critical risks across several categories that require immediate attention.

Key Metrics:

  • Total Repos: 91
  • Forked Repos: ~50+ (55%+)
  • Original Repos: ~40
  • Inactive Repos: 15+ (no activity for 1+ year)
  • Repositories with Missing Docs: 45+
  • Private Repos: 2-3

🚨 CRITICAL ISSUES (Priority 1)

1. Outdated & Abandoned Forks (17 repos)

Repository Last Activity Days Since Parent Repo Issue
n8n 2024-04-13 ~700 days n8n-io/n8n SEVERELY OUTDATED - 2 years behind
real-estate-ai 2023-07-11 ~950 days jharnakhatun2/real-estate-ai ~2.5 years old, never synced
awesome-llm-apps 2025-07-26 ~250 days Shubhamsaboo/awesome-llm-apps 9 months stale, losing updates
use-cases-data-science Unknown Unknown (Original) No recent activity
house-price-prediction Unknown Unknown (Original) Appears abandoned
DoRA Unknown Unknown (Original) Academic repo, likely unmaintained
ChatGPT-Custom-Instructions Unknown Unknown (Original) Outdated use (ChatGPT now evolving)

Risk Level: 🔴 CRITICAL

Recommendation:

  • Delete or unstar outdated forks not actively maintained
  • Sync critical forks (n8n) with upstream or remove
  • Archive repos abandoned >6 months without commits

2. Poor Documentation (45+ repositories)

Repos with Missing or Minimal READMEs:

  • agent-skills - Fork created 12 minutes ago, no documentation
  • appraiser-helper-bigquery-ai-solution - Empty description
  • House-price-prediction1 - No description
  • saudi_aqar_price_prediction - No description
  • -1, -2, -3 - Placeholder names, no description (3 repos)
  • Entire awesome-* collection - Inconsistent documentation

Risk Level: 🔴 CRITICAL - Blocks discoverability and collaboration

Recommendation:

  • Create README templates for all public repos
  • Add minimum required sections: What it does, Installation, Usage, Contributing
  • Use GitHub's auto-generated docs if manual docs aren't added

3. Meaningless Repository Names (6 repos)

Repo Name What It Is Issue
- HTML report templates Unindexable, impossible to find
-1 Property portfolio JS/CSS app Non-standard naming
-2 Market data aggregation platform Non-standard naming
-3 Property valuation tools Non-standard naming
v0-untitled-project- Real estate dashboard Temp name used in production
v0-untitled-project--1a Financial modeling platform Temp name used in production

Risk Level: 🔴 CRITICAL - Breaks GitHub searchability

Recommendation:

  • Rename immediately to descriptive names:
    • -html-report-templates
    • -1property-portfolio-manager
    • -2market-data-analytics
    • -3property-valuation-tools
    • v0-* → actual project names

⚠️ HIGH-PRIORITY ISSUES (Priority 2)

4. Recently Forked Repos with No Customization (5 repos)

Repo Created Parent Status Issue
agent-skills 12 min ago addyosmani/agent-skills Fork Just duplicated, no changes
claude-code 6 days ago chatgptprojects/clear-code Fork Minimal activity
claw-code 5 days ago ultraworkers/claw-code Fork Last push 5+ days old
appraiser-helper-bigquery-ai-solution Recent Unknown Fork? Empty description

Risk Level: 🟠 HIGH

**Recommendation...

This pull request was created from Copilot chat.

Copilot AI changed the title [WIP] Fix hardcoded repository reference in bump-go.sh fix: correct hardcoded repo reference in bump-go.sh Apr 6, 2026
Copilot AI requested a review from Moshbbab April 6, 2026 21:38
@Moshbbab Moshbbab marked this pull request as ready for review April 6, 2026 21:42
@Moshbbab Moshbbab merged commit 6e97c84 into trunk Apr 6, 2026
22 checks passed
@Moshbbab Moshbbab deleted the copilot/fix-bump-go-script-repo-reference branch April 6, 2026 21:43
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.

2 participants