Skip to content

Fix advisory search in Activity 2.1 - #128

Open
khlaifiabilel wants to merge 1 commit into
skills:mainfrom
khlaifiabilel:fix/advisory-search
Open

Fix advisory search in Activity 2.1#128
khlaifiabilel wants to merge 1 commit into
skills:mainfrom
khlaifiabilel:fix/advisory-search

Conversation

@khlaifiabilel

Copy link
Copy Markdown

Summary

  • use affects:follow-redirects to filter the Advisory Database results to advisories affecting the dependency
  • update the result explanation to describe advisory vulnerability and patched-version details accurately

Fixes #126

Tests

  • git diff --check
  • rendered .github/steps/2-dependabot-alerts.md with the GitHub Markdown API
  • verified the filtered Advisory Database URL and GitHub search-syntax documentation return HTTP 200
  • confirmed the live affects:follow-redirects search returns five advisories for the npm package

UI exercise flow was not run in a temporary repository.


This replaces #127, which closed automatically after the head fork was accidentally deleted.

Copilot AI review requested due to automatic review settings July 31, 2026 15:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Step 2 Dependabot lesson to make Activity 2.1’s Advisory Database search more accurate by using GitHub Advisory search syntax that filters to advisories affecting the follow-redirects package, and refreshes the explanatory text to better reflect what the advisory details represent.

Changes:

  • Switches the Activity 2.1 search query to affects:follow-redirects to reduce irrelevant advisory results.
  • Rewords the results explanation to focus on vulnerability details and patched versions.

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

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.

[Bug] search string in activity 2.1 leads to too many irrelevant results

2 participants