Bump prettier to 3.3.3 and reformat files

[twschiller]

What does this PR do?

• Bump prettier to 3.3.3 and run against all files
• See linked code

Discussion

• Do we need prettier as an npm dev dependency? Do people's IDEs use that if they have it configured (or does that functionality use npx)?

Future Work

• Create a job to automatically bump prettier and run over all files

For more information on our expectations for the PR process, see the
code review principles doc

[twschiller]

Main change is here

[codecov]

Codecov Report

Attention: Patch coverage is 62.96296% with 10 lines in your changes missing coverage. Please review.

Project coverage is 75.80%. Comparing base (8318d74) to head (666e944).
Report is 504 commits behind head on main.

Files with missing lines	Patch %	Lines
...browser-extension/src/testUtils/permissionsMock.ts	0.00%	4 Missing ⚠️
...ields/schemaFields/widgets/DatabaseCreateModal.tsx	0.00%	2 Missing ⚠️
...elds/schemaFields/widgets/varPopup/SourceLabel.tsx	0.00%	2 Missing ⚠️
...xtension/src/components/MarketplaceListingIcon.tsx	0.00%	1 Missing ⚠️
...ionConsole/pages/packageEditor/useSubmitPackage.ts	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9503      +/-   ##
==========================================
+ Coverage   74.24%   75.80%   +1.55%     
==========================================
  Files        1332     1422      +90     
  Lines       40817    42916    +2099     
  Branches     7634     7900     +266     
==========================================
+ Hits        30306    32531    +2225     
+ Misses      10511    10385     -126     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Playwright test results

 153 passed
 1 flaky
 2 skipped

Details

 Open report ↗︎
 156 tests across 51 suites
 10 minutes, 46 seconds
 666e944
 For more information on how to debug and view this report, see our readme

Flaky tests

msedge-setup › setup/unaffiliated.setup.ts › authenticate with unaffiliated user

Skipped tests

chrome › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options
msedge › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options

[twschiller]

💅

[grahamlangford]

Big fan of this prettier change

[twschiller]

Let's keep these sorted

[twschiller]

@fungairino @grahamlangford we'll want to consider workspace-specific prettierignore and gitignore files

[grahamlangford]

Do we need prettier as an npm dev dependency? Do people's IDEs use that if they have it configured (or does that functionality use npx)?

Exactly. Remember when we had to commit, add, commit in order to satisfy the prettier step in pre-commit? Making prettier a devDependency ensured that pre-commit and local format-on-save agreed

See: #7553

[github-actions]

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.