Skip to content

chore(deps): remove orphaned seed files keeping Dependabot alerts open#16235

Merged
davidkonigsberg merged 1 commit into
mainfrom
devin/1780575498-remove-orphan-lockfile
Jun 4, 2026
Merged

chore(deps): remove orphaned seed files keeping Dependabot alerts open#16235
davidkonigsberg merged 1 commit into
mainfrom
devin/1780575498-remove-orphan-lockfile

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented Jun 4, 2026

Description

Removes orphaned root-level files from seed/python-sdk/basic-auth-pw-omitted/ that still reference aiohttp 3.13.5, keeping Dependabot alerts #2080 and #2081 open despite the generator fix in #16230.

Changes Made

  • Deleted orphaned poetry.lock, pyproject.toml, README.md, reference.md, requirements.txt, and snippet.json from seed/python-sdk/basic-auth-pw-omitted/
  • The seed.yml config for this fixture only generates output into the with-wire-tests/ subfolder (which already has aiohttp 3.14.0). The root-level files were leftovers from a previous configuration and not managed by the seed tool.
  • Updated README.md generator (if applicable) — N/A

Testing

  • Verified seed.yml only declares outputFolder: with-wire-tests for basic-auth-pw-omitted
  • Confirmed with-wire-tests/ already has the correct aiohttp 3.14.0 constraint
  • No other fixture has this orphaned root-level + subfolder pattern

Link to Devin session: https://app.devin.ai/sessions/89af7d2e63e64a8c834cb8e8cf03969d


Open in Devin Review

The seed.yml config for basic-auth-pw-omitted only generates output into
the with-wire-tests/ subfolder. These root-level files were orphaned
from a previous configuration and still referenced aiohttp 3.13.5,
keeping Dependabot alerts #2080 and #2081 open despite the generator
fix in #16230.
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot requested a review from amckinney as a code owner June 4, 2026 12:18
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

Copy link
Copy Markdown
Contributor Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@davidkonigsberg davidkonigsberg merged commit e60e91f into main Jun 4, 2026
63 checks passed
@davidkonigsberg davidkonigsberg deleted the devin/1780575498-remove-orphan-lockfile branch June 4, 2026 12:27
devin-ai-integration Bot added a commit that referenced this pull request Jun 5, 2026
…E-2026-47266

Both Dependabot alerts #2081 (CVE-2026-47265, cross-origin redirect with
per-request cookies) and #2080 (CVE-2026-47266, deserialization of untrusted
data) reference seed/python-sdk/basic-auth-pw-omitted/poetry.lock which no
longer exists in the repository.

The aiohttp lower bound was already bumped to >=3.14.0 in PR #16230 and all
remaining seed poetry.lock files contain aiohttp 3.14.0 (the patched version).
The orphaned manifest file was removed in PR #16235.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant