Skip to content

update fetch -> web/fetch everywhere #532

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
aaronpowell merged 7 commits into from
Jan 12, 2026
Merged

update fetch -> web/fetch everywhere #532

aaronpowell merged 7 commits into from
Jan 12, 2026

Conversation

@Jrc356
Copy link
Contributor

@Jrc356 Jrc356 commented Jan 3, 2026 •
edited
Loading

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • My contribution adds a new instruction, prompt, or chat mode file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, or chat mode with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description

My vscode yelled that fetch has been moved to web/fetch instead so was hoping to push this upstream as well.

Type of Contribution

  • New instruction file.
  • New prompt file.
  • New chat mode file.
  • New collection file.
  • Update to existing instruction, prompt, chat mode, or collection.
  • Other (please specify):

Additional Notes

By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

@Jrc356
documentation-writer: fetch -> web/fetch
6500ed2 
Updated tools list to include 'web/fetch' instead of 'fetch'.
Copilot AI review requested due to automatic review settings January 3, 2026 22:34
Copilot AI reviewed Jan 3, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates a single tool reference from 'fetch' to 'web/fetch' in the documentation-writer.prompt.md file, based on VS Code guidance that the fetch tool has been moved to web/fetch.

Key Changes:

  • Updated the tools array in the front matter from 'fetch' to 'web/fetch'

@codemillmatt codemillmatt self-requested a review January 9, 2026 18:24
@codemillmatt
Copy link
Contributor

codemillmatt commented Jan 9, 2026

Thanks for this one @Jrc356 . We probably should do a wholesale find/replace. 😅

Tagging @aaronpowell for his thoughts first.

@aaronpowell
Copy link
Contributor

aaronpowell commented Jan 11, 2026

Yeah, looks like we have a lot of legacy 'fetch' that should be updated in a bulk PR.

@Jrc356 Jrc356 marked this pull request as draft January 12, 2026 19:00
@Jrc356
Copy link
Contributor Author

Jrc356 commented Jan 12, 2026

sounds good will update everything! gonna draft in the meantime (I've already borked newlines by accident so gotta clean a bit 😅)

Jrc356
Jrc356 commented Jan 12, 2026
View reviewed changes
Jrc356
Jrc356 commented Jan 12, 2026
View reviewed changes
@Jrc356 Jrc356 changed the title documentation-writer: fetch -> web/fetch update fetch -> web/fetch everywhere Jan 12, 2026
@Jrc356 Jrc356 marked this pull request as ready for review January 12, 2026 19:12
@Jrc356
Copy link
Contributor Author

Jrc356 commented Jan 12, 2026

alrighty, @codemillmatt @aaronpowell we should be good to go here! I noticed while doing some spot checking that there were other tools in other files that are now out of date as well. Unsure how this warning is triggered but perhaps there's a spec that could be looked at to automate keeping tool lists updated?

If I find some time, I might poke around but food for you thought otherwise :)

@aaronpowell
Copy link
Contributor

aaronpowell commented Jan 12, 2026

alrighty, @codemillmatt @aaronpowell we should be good to go here! I noticed while doing some spot checking that there were other tools in other files that are now out of date as well. Unsure how this warning is triggered but perhaps there's a spec that could be looked at to automate keeping tool lists updated?

If I find some time, I might poke around but food for you thought otherwise :)

yeah, there's quite a few that are out of date, trying to stay on top of everything is basically my full-time job 🤣

@aaronpowell aaronpowell merged commit 4b232e4 into github:main Jan 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@aaronpowell aaronpowell aaronpowell approved these changes

@codemillmatt codemillmatt Awaiting requested review from codemillmatt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Jrc356 @codemillmatt @aaronpowell