Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 25, 2026

Addresses review feedback requesting user visibility when version resolution falls back from "latest" to the hardcoded default.

Changes:

  • Added warning messages to stderr when upstream version fetch fails (network/timeout) or version extraction fails
  • Warnings specify the fallback version being used and the reason for fallback

Implementation:

# Two distinct failure modes now surface to users:
if upstream_script=$(curl -fsSL --connect-timeout 10 "$AIKIDO_UPSTREAM_SCRIPT" 2>/dev/null); then
    if [ -n "$extracted_version" ]; then
        echo "$extracted_version"
        return 0
    fi
    echo "Warning: Could not extract version from upstream script, using fallback version $FALLBACK_VERSION" >&2
else
    echo "Warning: Failed to fetch upstream version (network or connection timeout), using fallback version $FALLBACK_VERSION" >&2
fi

Users now understand when and why they're not getting the latest version when version: "latest" is specified.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: Datacom-Ayden <221437541+Datacom-Ayden@users.noreply.github.com>
Copilot AI changed the title [WIP] Add automated version management for Aikido pre-commit scanner Add fallback logging to aikido-precommit version resolution Jan 25, 2026
Copilot AI requested a review from Datacom-Ayden January 25, 2026 05:37
@Datacom-Ayden Datacom-Ayden marked this pull request as ready for review January 25, 2026 05:38
@Datacom-Ayden Datacom-Ayden merged commit bb7a32b into feat/Aikido-version-mgmt Jan 25, 2026
@Datacom-Ayden Datacom-Ayden deleted the copilot/sub-pr-7 branch January 25, 2026 05:38
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