fix: correct hardcoded repo reference in bump-go.sh#8
Merged
Conversation
…mp-go.sh Agent-Logs-Url: https://github.com/Moshbbab/cli/sessions/fb5ec5e1-941f-4202-8d38-0ae715c65c5e Co-authored-by: Moshbbab <132464244+Moshbbab@users.noreply.github.com>
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
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.
The
bump-go.shscript had a hardcodedcli/clirepo 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.shline 93:--repo cli/cli→--repo Moshbbab/cliOriginal prompt
Issue
The
.github/workflows/scripts/bump-go.shscript 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. Thegh search prscommand cannot find existing PRs in the correct repository.Change Required
Replace
cli/cliwithMoshbbab/clion 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:
🚨 CRITICAL ISSUES (Priority 1)
1. Outdated & Abandoned Forks (17 repos)
n8nreal-estate-aiawesome-llm-appsuse-cases-data-sciencehouse-price-predictionDoRAChatGPT-Custom-InstructionsRisk Level: 🔴 CRITICAL
Recommendation:
2. Poor Documentation (45+ repositories)
Repos with Missing or Minimal READMEs:
agent-skills- Fork created 12 minutes ago, no documentationappraiser-helper-bigquery-ai-solution- Empty descriptionHouse-price-prediction1- No descriptionsaudi_aqar_price_prediction- No description-1,-2,-3- Placeholder names, no description (3 repos)awesome-*collection - Inconsistent documentationRisk Level: 🔴 CRITICAL - Blocks discoverability and collaboration
Recommendation:
3. Meaningless Repository Names (6 repos)
--1-2-3v0-untitled-project-v0-untitled-project--1aRisk Level: 🔴 CRITICAL - Breaks GitHub searchability
Recommendation:
-→html-report-templates-1→property-portfolio-manager-2→market-data-analytics-3→property-valuation-toolsv0-*→ actual project names4. Recently Forked Repos with No Customization (5 repos)
agent-skillsclaude-codeclaw-codeappraiser-helper-bigquery-ai-solutionRisk Level: 🟠 HIGH
**Recommendation...
This pull request was created from Copilot chat.