Skip to content

feat: migrate browser data from the legacy web origin to web.phcode.dev - #3130

Merged
abose merged 3 commits into
mainfrom
ai
Aug 27, 2026
Merged

feat: migrate browser data from the legacy web origin to web.phcode.dev#3130
abose merged 3 commits into
mainfrom
ai

test: make the MigrateAssist round trip seed its own project

9917d4a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 27, 2026 in 38s

Quality Gate passed

Issues
17 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 106 in src/extensionsIntegrated/MigrateAssist/migrator.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-UOVmC7munXgIE&open=AaAq5W-UOVmC7munXgIE&pullRequest=3130

Check warning on line 278 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIS&open=AaAq5XAoOVmC7munXgIS&pullRequest=3130

Check failure on line 211 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 27 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIP&open=AaAq5XAoOVmC7munXgIP&pullRequest=3130

Check warning on line 91 in src/extensionsIntegrated/MigrateAssist/constants.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-hOVmC7munXgII&open=AaAq5W-hOVmC7munXgII&pullRequest=3130

Check failure on line 244 in src/extensionsIntegrated/MigrateAssist/migrator.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-UOVmC7munXgIH&open=AaAq5W-UOVmC7munXgIH&pullRequest=3130

Check warning on line 82 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIM&open=AaAq5XAoOVmC7munXgIM&pullRequest=3130

Check warning on line 234 in src/extensionsIntegrated/MigrateAssist/migrator.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_applyPhStore' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-UOVmC7munXgIG&open=AaAq5W-UOVmC7munXgIG&pullRequest=3130

Check failure on line 157 in src/extensionsIntegrated/MigrateAssist/migrator.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest functions more than 5 levels deep. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-UOVmC7munXgIF&open=AaAq5W-UOVmC7munXgIF&pullRequest=3130

Check warning on line 110 in src/extensionsIntegrated/MigrateAssist/sunset-dialog.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-rOVmC7munXgIL&open=AaAq5W-rOVmC7munXgIL&pullRequest=3130

Check warning on line 2930 in src/nls/root/strings.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unnecessary escape character: \'. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W4iOVmC7munXgID&open=AaAq5W4iOVmC7munXgID&pullRequest=3130

Check warning on line 154 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIO&open=AaAq5XAoOVmC7munXgIO&pullRequest=3130

Check warning on line 2930 in src/nls/root/strings.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unnecessary escape character: \'. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W4iOVmC7munXgIC&open=AaAq5W4iOVmC7munXgIC&pullRequest=3130

Check warning on line 258 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIR&open=AaAq5XAoOVmC7munXgIR&pullRequest=3130

Check warning on line 157 in src/extensionsIntegrated/MigrateAssist/constants.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function 'isMigrationSupportedBrowser' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-hOVmC7munXgIJ&open=AaAq5W-hOVmC7munXgIJ&pullRequest=3130

Check warning on line 231 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIQ&open=AaAq5XAoOVmC7munXgIQ&pullRequest=3130

Check warning on line 146 in src/migrateAssist.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5XAoOVmC7munXgIN&open=AaAq5XAoOVmC7munXgIN&pullRequest=3130

Check warning on line 67 in src/extensionsIntegrated/MigrateAssist/sunset-dialog.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not call `Array#push()` multiple times. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AaAq5W-rOVmC7munXgIK&open=AaAq5W-rOVmC7munXgIK&pullRequest=3130
View more details on SonarQubeCloud