#7377: fix local UiPath process configuration and page editor detection

[twschiller]

What does this PR do?

• Closes UiPath assistant not connecting in Page Editor #7377
• Removes unnecessary isDevToolsPage check that was failing - includes @fregante's #7378: Fix page editor detection #7391 PR to fix bugs in dev tools page checks
• Fixes options generator

Remaining Work

• Determine why isDevToolsPage was failing, so we can verify other uses of the method
  • I think the wrong check just stuck in. IsDevToolsPage would match "devtools.html", which is the devtools background page, not pageEditor.html, which is the panel we add to the Dev Tools
  • The other uses look OK:

    • pixiebrix-extension/src/contrib/google/sheets/core/useCurrentOrigin.ts

      Line 47 in c2a7420

      isDevToolsPage() ||

    • pixiebrix-extension/src/telemetry/BackgroundLogger.ts

      Line 75 in c2a7420

      async error(error: unknown, data: JsonObject): Promise<void> {

Discussion

• Decide if we want to drop the API call to the control room to fetch the inputs. The comment here where using the brick introduces an unnecessary integration still applies:

  pixiebrix-extension/src/contrib/uipath/LocalProcessOptions.tsx

  Line 105 in c2a7420

  // FIXME: this integration use is options-only. As-is this will create an integration entry in the background.

Demo

• https://www.loom.com/share/379261ebe6d546acb35095ac95cf62fc?sid=b87fc2c6-7e1a-4adc-9bcf-62bb0e791b92

Checklist

• Add tests: manually tested on Windows and Mac; Jest testing the dev tools detection issue wouldn't be much value because it's so reliant on the mocking. Will add to regression testing sheet
• New files added to src/tsconfig.strictNullChecks.json (if possible): N/A
• Designate a primary reviewer: @grahamlangford

[codecov]

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (ad8cd60) 72.50% compared to head (fdfa4d6) 72.50%.

Files	Patch %	Lines
src/contrib/uipath/localProcess.ts	50.00%	2 Missing ⚠️
src/utils/expectContext.ts	60.00%	2 Missing ⚠️
src/pageEditor/hooks/useCurrentUrl.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7378      +/-   ##
==========================================
- Coverage   72.50%   72.50%   -0.01%     
==========================================
  Files        1207     1207              
  Lines       37663    37651      -12     
  Branches     7062     7060       -2     
==========================================
- Hits        27306    27297       -9     
+ Misses      10357    10354       -3     

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

[fregante]

Note that #7391 was already reviewed and merged into this PR

[github-actions]

When the PR is merged, the first loom link found on this PR will be posted to #sprint-demo on Slack. Do not edit this comment manually.

[twschiller]

@grahamlangford this will need to be cherry-picked into 1.8.7