chore: remove basic-ftp override — lockfile resolution is sufficient

[Copilot]

The pnpm.overrides exception for basic-ftp is unnecessary. get-uri already declares basic-ftp@^5.0.2, which naturally satisfies 5.2.0. The lockfile was simply frozen to 5.0.5 from a previous install; updating the pinned version directly is the correct fix.

Changes

• package.json: Remove the pnpm.overrides block for basic-ftp
• pnpm-lock.yaml: Remove the overrides section; basic-ftp resolves to 5.2.0 via the existing ^5.0.2 range in get-uri

Dependency chain for reference:

live-plugin-manager → proxy-agent → pac-proxy-agent → get-uri → basic-ftp@^5.0.2 → resolves to 5.2.0

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[JohnMcLear]

@copilot The problem with this pull request is that it creates an exception for this package. I don't like it, shouldn't the upstream fix be implemented IE in live-plugin-manager → proxy-agent → pac-proxy-agent → get-uri or have we fixed the verison of one of those which is why it's not landed?

[SamTV12345]

@copilot The problem with this pull request is that it creates an exception for this package. I don't like it, shouldn't the upstream fix be implemented IE in live-plugin-manager → proxy-agent → pac-proxy-agent → get-uri or have we fixed the verison of one of those which is why it's not landed?

Is AI now contributing itself to our project. That's funny :) I don't really get what Copilot is trying to do.

[Copilot]

@copilot The problem with this pull request is that it creates an exception for this package. I don't like it, shouldn't the upstream fix be implemented IE in live-plugin-manager → proxy-agent → ...

The override has been removed in d6be191. get-uri already declares basic-ftp@^5.0.2, which naturally satisfies 5.2.0 — no upstream package needed changing. The lockfile was simply frozen to 5.0.5 from an older install. The fix was to drop the pnpm.overrides block entirely and let the existing semver range resolve to 5.2.0 on its own.

[SamTV12345]

@copilot The problem with this pull request is that it creates an exception for this package. I don't like it, shouldn't the upstream fix be implemented IE in live-plugin-manager → proxy-agent → ...

The override has been removed in d6be191. get-uri already declares basic-ftp@^5.0.2, which naturally satisfies 5.2.0 — no upstream package needed changing. The lockfile was simply frozen to 5.0.5 from an older install. The fix was to drop the pnpm.overrides block entirely and let the existing semver range resolve to 5.2.0 on its own.

That makes sense. Thanks!

[Copilot]

@SamTV12345 The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 71439a89-dc3c-4007-9b23-b4459a49a5a3

[Copilot]

@SamTV12345 The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 71439a89-dc3c-4007-9b23-b4459a49a5a3

[Copilot]

@SamTV12345 The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 71439a89-dc3c-4007-9b23-b4459a49a5a3

[Copilot]

@SamTV12345 The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 71439a89-dc3c-4007-9b23-b4459a49a5a3

[Copilot]

@SamTV12345 The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 71439a89-dc3c-4007-9b23-b4459a49a5a3

[Copilot]

@SamTV12345 The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 71439a89-dc3c-4007-9b23-b4459a49a5a3

[SamTV12345]

LGTM

[JohnMcLear]

I didn't like it, it's unnecessary surgery that will be patched upstream and exposed 0 attack vectors to the software. Was unnecessary noise I should have probably just ignored :P