[#1656] Updated the namespace for the installer.#1721
Conversation
WalkthroughThis change updates the namespace prefix from Changes
Sequence Diagram(s)sequenceDiagram
participant Composer
participant App
participant Command
participant Utils
participant Prompts
participant Handlers
Composer->>App: Autoload classes using DrevOps\VortexInstaller namespace
App->>Command: Use InstallCommand (new namespace)
Command->>Prompts: Use PromptManager (new namespace)
Prompts->>Handlers: Instantiate handlers (new namespace)
Handlers->>Utils: Use utility classes (new namespace)
Possibly related issues
Possibly related PRs
Suggested labels
Poem
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (1)
.vortex/installer/composer.lockis excluded by!**/*.lock
📒 Files selected for processing (55)
.vortex/installer/composer.json(1 hunks).vortex/installer/src/Command/InstallCommand.php(2 hunks).vortex/installer/src/Prompts/Handlers/AbstractHandler.php(1 hunks).vortex/installer/src/Prompts/Handlers/AssignAuthorPr.php(1 hunks).vortex/installer/src/Prompts/Handlers/CiProvider.php(1 hunks).vortex/installer/src/Prompts/Handlers/CodeProvider.php(1 hunks).vortex/installer/src/Prompts/Handlers/DatabaseDownloadSource.php(1 hunks).vortex/installer/src/Prompts/Handlers/DatabaseImage.php(1 hunks).vortex/installer/src/Prompts/Handlers/DependencyUpdatesProvider.php(1 hunks).vortex/installer/src/Prompts/Handlers/DeployType.php(1 hunks).vortex/installer/src/Prompts/Handlers/Domain.php(1 hunks).vortex/installer/src/Prompts/Handlers/GithubRepo.php(1 hunks).vortex/installer/src/Prompts/Handlers/GithubToken.php(1 hunks).vortex/installer/src/Prompts/Handlers/HandlerInterface.php(1 hunks).vortex/installer/src/Prompts/Handlers/HostingProvider.php(1 hunks).vortex/installer/src/Prompts/Handlers/Internal.php(1 hunks).vortex/installer/src/Prompts/Handlers/LabelMergeConflictsPr.php(1 hunks).vortex/installer/src/Prompts/Handlers/MachineName.php(1 hunks).vortex/installer/src/Prompts/Handlers/ModulePrefix.php(1 hunks).vortex/installer/src/Prompts/Handlers/Name.php(1 hunks).vortex/installer/src/Prompts/Handlers/Org.php(1 hunks).vortex/installer/src/Prompts/Handlers/OrgMachineName.php(1 hunks).vortex/installer/src/Prompts/Handlers/PreserveDocsOnboarding.php(1 hunks).vortex/installer/src/Prompts/Handlers/PreserveDocsProject.php(1 hunks).vortex/installer/src/Prompts/Handlers/Profile.php(1 hunks).vortex/installer/src/Prompts/Handlers/ProvisionType.php(1 hunks).vortex/installer/src/Prompts/Handlers/Services.php(1 hunks).vortex/installer/src/Prompts/Handlers/Theme.php(1 hunks).vortex/installer/src/Prompts/Handlers/ThemeRunner.php(1 hunks).vortex/installer/src/Prompts/Handlers/Webroot.php(1 hunks).vortex/installer/src/Prompts/PromptManager.php(4 hunks).vortex/installer/src/Utils/Composer.php(1 hunks).vortex/installer/src/Utils/Config.php(1 hunks).vortex/installer/src/Utils/Converter.php(1 hunks).vortex/installer/src/Utils/Downloader.php(1 hunks).vortex/installer/src/Utils/Env.php(1 hunks).vortex/installer/src/Utils/File.php(1 hunks).vortex/installer/src/Utils/Git.php(1 hunks).vortex/installer/src/Utils/Strings.php(1 hunks).vortex/installer/src/Utils/Tui.php(1 hunks).vortex/installer/src/Utils/Validator.php(1 hunks).vortex/installer/src/app.php(1 hunks).vortex/installer/tests/Functional/FunctionalTestCase.php(1 hunks).vortex/installer/tests/Functional/InstallTest.php(1 hunks).vortex/installer/tests/Traits/TuiTrait.php(1 hunks).vortex/installer/tests/Unit/ConverterTest.php(1 hunks).vortex/installer/tests/Unit/EnvTest.php(1 hunks).vortex/installer/tests/Unit/FileTest.php(4 hunks).vortex/installer/tests/Unit/GitTest.php(1 hunks).vortex/installer/tests/Unit/PromptManagerTest.php(1 hunks).vortex/installer/tests/Unit/SelfTest.php(1 hunks).vortex/installer/tests/Unit/StringsTest.php(1 hunks).vortex/installer/tests/Unit/TuiTest.php(1 hunks).vortex/installer/tests/Unit/UnitTestCase.php(1 hunks).vortex/installer/tests/Unit/ValidatorTest.php(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
.vortex/installer/src/Utils/File.php (1)
Learnt from: AlexSkrypnyk
PR: drevops/vortex#1693
File: .vortex/installer/src/Prompts/Handlers/Internal.php:52-52
Timestamp: 2025-06-01T08:10:15.903Z
Learning: The File utility class in .vortex/installer/src/Utils/File.php extends AlexSkrypnyk\File\File, and methods like collapseRepeatedEmptyLines() are available from this external parent class dependency.
🧬 Code Graph Analysis (27)
.vortex/installer/src/Prompts/Handlers/CiProvider.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/tests/Traits/TuiTrait.php (1)
.vortex/installer/src/Utils/Tui.php (1)
Tui(16-238)
.vortex/installer/src/Prompts/Handlers/Org.php (2)
.vortex/installer/src/Utils/Composer.php (1)
Composer(7-36).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/ModulePrefix.php (2)
.vortex/installer/src/Utils/Converter.php (1)
Converter(9-17).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/AbstractHandler.php (2)
.vortex/installer/src/Utils/Config.php (1)
Config(14-124).vortex/installer/src/Utils/Converter.php (1)
Converter(9-17)
.vortex/installer/src/Prompts/Handlers/CodeProvider.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/PreserveDocsProject.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/tests/Unit/TuiTest.php (1)
.vortex/installer/src/Utils/Tui.php (1)
Tui(16-238)
.vortex/installer/src/Prompts/Handlers/HostingProvider.php (2)
.vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/GithubToken.php (1)
.vortex/installer/src/Utils/Env.php (1)
Env(7-124)
.vortex/installer/src/Prompts/Handlers/DatabaseDownloadSource.php (2)
.vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/Theme.php (3)
.vortex/installer/src/Utils/Converter.php (1)
Converter(9-17).vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/DeployType.php (3)
.vortex/installer/src/Utils/Converter.php (1)
Converter(9-17).vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/tests/Unit/FileTest.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/Webroot.php (3)
.vortex/installer/src/Utils/Composer.php (1)
Composer(7-36).vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/ProvisionType.php (2)
.vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/Domain.php (3)
.vortex/installer/src/Utils/Converter.php (1)
Converter(9-17).vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/Services.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/DependencyUpdatesProvider.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/MachineName.php (3)
.vortex/installer/src/Utils/Composer.php (1)
Composer(7-36).vortex/installer/src/Utils/Converter.php (1)
Converter(9-17).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/tests/Functional/FunctionalTestCase.php (6)
.vortex/installer/src/Utils/File.php (1)
File(10-120).vortex/installer/src/Prompts/Handlers/Internal.php (1)
Internal(12-129).vortex/installer/src/Command/InstallCommand.php (1)
InstallCommand(26-433).vortex/installer/tests/Unit/UnitTestCase.php (1)
UnitTestCase(20-61).vortex/installer/src/Utils/Config.php (1)
Config(14-124).vortex/installer/src/Utils/Env.php (1)
Env(7-124)
.vortex/installer/src/Prompts/Handlers/Profile.php (2)
.vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/Name.php (2)
.vortex/installer/src/Utils/Composer.php (1)
Composer(7-36).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/GithubRepo.php (1)
.vortex/installer/src/Utils/Git.php (1)
Git(11-113)
.vortex/installer/src/Prompts/Handlers/DatabaseImage.php (2)
.vortex/installer/src/Utils/Env.php (1)
Env(7-124).vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/tests/Unit/UnitTestCase.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
.vortex/installer/src/Prompts/Handlers/PreserveDocsOnboarding.php (1)
.vortex/installer/src/Utils/File.php (1)
File(10-120)
⏰ Context from checks skipped due to timeout of 90000ms (8)
- GitHub Check: build (1)
- GitHub Check: build (0)
- GitHub Check: vortex-test-deployment (1)
- GitHub Check: vortex-test-deployment (0)
- GitHub Check: vortex-test-workflow (3)
- GitHub Check: vortex-test-workflow (1)
- GitHub Check: vortex-test-workflow (2)
- GitHub Check: vortex-test-workflow (0)
🔇 Additional comments (78)
.vortex/installer/src/Prompts/Handlers/CiProvider.php (1)
5-7: Consistent namespace migration
The namespace and import for theFileutility have been correctly updated toDrevOps\VortexInstaller, aligning with the new PSR-4 mapping and ensuring references resolve properly..vortex/installer/src/Utils/File.php (1)
5-5: Namespace updated toDrevOps\VortexInstaller\Utils
The PSR-4 declaration now matches the directory structure and Composer autoload configuration..vortex/installer/src/Utils/Env.php (1)
5-5: Namespace updated toDrevOps\VortexInstaller\Utils
PSR-4 declaration is aligned with the updated installer namespace..vortex/installer/src/Utils/Downloader.php (1)
5-5: Namespace updated toDrevOps\VortexInstaller\Utils
The namespace now reflects the new module prefix and matches the autoload mapping..vortex/installer/src/Utils/Composer.php (1)
5-5: Namespace updated toDrevOps\VortexInstaller\Utils
Declaration correctly reflects the new PSR-4 namespace for the Composer helper..vortex/installer/src/Utils/Converter.php (2)
5-5: Namespace update is correct.
The PSR-4 namespace now aligns with the folder structure and the newVortexInstallerprefix.
1-18:Details
✅ Verification successful
Verify PSR-4 autoload mapping.
Double-check thatcomposer.json’s"autoload"section has been updated to mapDrevOps\\VortexInstaller\\Utils\\(and the baseDrevOps\\VortexInstaller\\) tosrc/so these classes will autoload correctly.
🏁 Script executed:
#!/bin/bash # Verify no lingering Installer namespace in composer.json rg -n '"DrevOps\\\\Installer"' composer.jsonLength of output: 46
I couldn’t find any
"DrevOps\\Installer"entries in composer.json, but we need to confirm that your PSR-4 mappings includeDrevOps\\VortexInstaller\\. Let’s dump theautoload.psr-4section to be sure:
🏁 Script executed:
#!/bin/bash # Display PSR-4 autoload mappings to verify VortexInstaller namespace jq '.autoload["psr-4"]' composer.jsonLength of output: 44
Let’s locate the correct composer.json to inspect its PSR-4 mappings:
🏁 Script executed:
#!/bin/bash # Find all composer.json files in the repo find . -type f -name composer.jsonLength of output: 17365
Let’s inspect the PSR-4 mappings in the installer’s composer.json:
🏁 Script executed:
#!/bin/bash # Show PSR-4 autoload mappings in .vortex/installer/composer.json jq '.autoload["psr-4"]' ./.vortex/installer/composer.jsonLength of output: 102
PSR-4 autoload mapping verified
The installer’s composer.json maps"DrevOps\\VortexInstaller\\"→"src/", which coversDrevOps\VortexInstaller\Utils\. No changes needed..vortex/installer/tests/Unit/SelfTest.php (1)
5-5: Namespace update in test is correct.
The test class is now inDrevOps\VortexInstaller\Tests\Unitand matches the new PSR-4 mapping..vortex/installer/src/Utils/Tui.php (1)
5-5: Namespace update is correct.
Tuinow lives underDrevOps\VortexInstaller\Utilsand matches the updated mapping..vortex/installer/src/app.php (1)
10-10: Import updated forInstallCommand.
The use statement now referencesDrevOps\VortexInstaller\Command\InstallCommand, matching the renamed class location..vortex/installer/src/Utils/Git.php (1)
5-5: Namespace update is correct.
TheGitutility class now uses theDrevOps\VortexInstaller\Utilsnamespace as intended..vortex/installer/tests/Traits/TuiTrait.php (1)
5-7: Consistent namespace and import updated
The namespace and theTuiimport have been correctly updated toDrevOps\VortexInstaller. This change aligns with the PSR-4 structure and keeps the test trait in sync with the refactored installer namespace..vortex/installer/src/Utils/Strings.php (1)
5-5: Namespace updated toDrevOps\VortexInstaller\Utils
The namespace declaration now matches the updated PSR-4 autoload mapping. No additional changes needed..vortex/installer/src/Prompts/Handlers/AssignAuthorPr.php (1)
5-5: Namespace updated toDrevOps\VortexInstaller\Prompts\Handlers
The class namespace correctly reflects the new installer prefix, ensuring autoload resolution remains intact..vortex/installer/tests/Unit/StringsTest.php (1)
5-9: Namespace and import updated forStringsTest
Both the test namespace and the import ofStringshave been updated toDrevOps\VortexInstaller. This ensures the unit tests target the refactored utility class under the new namespace..vortex/installer/src/Utils/Validator.php (1)
5-12: Namespace and docblock@packageannotation updated
Updated the namespace and the@packagePHPDoc tag toDrevOps\VortexInstaller. These changes keep the class aligned with the updated PSR-4 configuration and documentation consistency..vortex/installer/src/Utils/Config.php (2)
5-5: Namespace updated to reflect refactoring.
Changed fromDrevOps\Installer\UtilstoDrevOps\VortexInstaller\Utils, aligning with the new installer namespace.
12-12: Docblock@packageannotation adjusted.
Updated toDrevOps\VortexInstallerto match the revised namespace..vortex/installer/src/Prompts/Handlers/LabelMergeConflictsPr.php (1)
5-5: Namespace aligned with the new installer prefix.
Updated toDrevOps\VortexInstaller\Prompts\Handlers, consistent with adjacent handler classes..vortex/installer/src/Prompts/Handlers/ThemeRunner.php (1)
5-5: Namespace alignment is correct.
Now underDrevOps\VortexInstaller\Prompts\Handlers, matching the rest of the module..vortex/installer/composer.json (2)
46-46: PSR-4 autoload mapping updated.
MapsDrevOps\VortexInstaller\tosrc/, matching the namespace refactor.
51-51: Dev autoload mapping updated.
MapsDrevOps\VortexInstaller\Tests\totests, ensuring test classes load under the new namespace..vortex/installer/src/Prompts/Handlers/HandlerInterface.php (1)
5-13: Namespace and package annotation updated.
Revised toDrevOps\VortexInstaller\Prompts\Handlersin both thenamespacedeclaration and the@packagedocblock..vortex/installer/src/Prompts/Handlers/Services.php (1)
5-7: Namespace and import updated correctly
The namespace declaration andFileutility import have been changed toDrevOps\VortexInstaller. Ensure that your PSR-4 autoload mapping incomposer.jsonreflects this new prefix..vortex/installer/src/Prompts/Handlers/Profile.php (1)
5-8: Profile handler namespace and imports aligned
The namespace and bothEnvandFileutility imports have been updated to useDrevOps\VortexInstaller. Confirm that the autoload section incomposer.jsonhas been updated to match..vortex/installer/src/Prompts/Handlers/Org.php (1)
5-8: Org handler namespace and imports updated
The namespace and imports forComposerandFilehave been switched toDrevOps\VortexInstaller. Verify that PSR-4 autoload entries incomposer.jsoncover these new paths..vortex/installer/src/Prompts/Handlers/HostingProvider.php (1)
5-8: HostingProvider namespace and imports refreshed
The namespace and utility imports (Env,File) are now underDrevOps\VortexInstaller. Please double-check yourcomposer.jsonPSR-4 settings to ensure this package structure is properly mapped..vortex/installer/tests/Unit/TuiTest.php (1)
5-9: Test namespace and import updated for Tui
The test class namespace andTuiimport have been updated toDrevOps\VortexInstaller. Make sure that the test suite configuration and autoloader are aware of this change so tests continue to run..vortex/installer/src/Prompts/Handlers/ProvisionType.php (1)
5-8: Updated namespace and imports are correct. Files are now underDrevOps\VortexInstaller\Prompts\Handlersand importEnvandFilefromDrevOps\VortexInstaller\Utils. This aligns with the PSR-4 autoload configuration..vortex/installer/tests/Unit/ConverterTest.php (2)
5-5: Namespace updated correctly. Now usingDrevOps\VortexInstaller\Tests\Unit.
9-9: Import path updated.Converteris accurately imported fromDrevOps\VortexInstaller\Utils..vortex/installer/src/Prompts/Handlers/Domain.php (1)
5-9: Namespace and imports updated properly.Domainnow lives inDrevOps\VortexInstaller\Prompts\Handlersand pulls inConverter,Env, andFilefrom the updated utils namespace..vortex/installer/src/Prompts/Handlers/Name.php (1)
5-8: Namespace and imports updated. TheNamehandler is correctly namespaced underDrevOps\VortexInstaller\Prompts\Handlersand importsComposerandFilefromDrevOps\VortexInstaller\Utils..vortex/installer/tests/Unit/GitTest.php (2)
5-5: Namespace update correct. Now underDrevOps\VortexInstaller\Tests\Unit.
9-9: Import path updated.Gitis imported from the newDrevOps\VortexInstaller\Utilsnamespace..vortex/installer/src/Prompts/Handlers/AbstractHandler.php (1)
5-8:Details
✅ Verification successful
LGTM: Namespace refactoring correctly implemented.
The namespace and import statements have been properly updated from
DrevOps\InstallertoDrevOps\VortexInstaller. The changes are consistent and maintain the same class structure.Please verify that all references to the old namespace have been updated throughout the codebase:
🏁 Script executed:
#!/bin/bash # Description: Check for any remaining references to the old DrevOps\Installer namespace # Expected: No matches should be found echo "Searching for old namespace references..." rg "DrevOps\\\\Installer" --type php rg "DrevOps\\\\Installer" --type json echo "Searching for old use statements..." rg "use DrevOps\\\\Installer" --type php echo "Checking for old namespace declarations..." rg "namespace DrevOps\\\\Installer" --type phpLength of output: 431
No leftover
DrevOps\Installerreferences
Searches forDrevOps\Installerin both PHP and JSON files returned no matches. The namespace refactoring toDrevOps\VortexInstalleris fully applied and consistent across the codebase. Approving changes..vortex/installer/src/Prompts/Handlers/DatabaseDownloadSource.php (1)
5-8: LGTM: Consistent namespace refactoring.The namespace and import statements have been correctly updated to use the new
DrevOps\VortexInstallerprefix, maintaining consistency with the overall refactoring effort..vortex/installer/src/Prompts/Handlers/Internal.php (1)
5-10: LGTM: Namespace refactoring properly applied.The namespace declaration and all utility class imports have been correctly updated to use the new
DrevOps\VortexInstallernamespace prefix. The implementation remains functionally unchanged..vortex/installer/src/Prompts/Handlers/OrgMachineName.php (1)
5-8: LGTM: Namespace update correctly implemented.The namespace and import statements have been properly updated to align with the new
DrevOps\VortexInstallernamespace structure. No functional changes were made to the class..vortex/installer/src/Prompts/Handlers/DeployType.php (1)
5-9: Namespace and imports updated correctly.
All occurrences ofDrevOps\Installerhave been changed toDrevOps\VortexInstaller, and the utility imports (Converter,Env,File) point to the correct new namespace..vortex/installer/src/Prompts/Handlers/ModulePrefix.php (1)
5-8: Namespace and imports updated correctly.
The class namespace and utility imports (Converter,File) now useDrevOps\VortexInstaller, matching the new module structure..vortex/installer/src/Prompts/Handlers/GithubToken.php (1)
5-7: Namespace and import updated correctly.
TheEnvutility is now imported fromDrevOps\VortexInstaller\Utils, aligning with the refactored namespace..vortex/installer/src/Prompts/Handlers/GithubRepo.php (1)
5-7: Namespace and import updated correctly.
TheGitutility import has been revised toDrevOps\VortexInstaller\Utils\Git, ensuring consistency with the new namespace..vortex/installer/src/Prompts/Handlers/MachineName.php (1)
5-9: Namespace and imports updated correctly.
All utility imports (Composer,Converter,File) and the namespace declaration use the newDrevOps\VortexInstallerprefix..vortex/installer/src/Prompts/Handlers/CodeProvider.php (2)
5-5: Updated namespace to VortexInstaller.
Changed fromDrevOps\Installer\Prompts\HandlerstoDrevOps\VortexInstaller\Prompts\Handlersto match the new module prefix.
7-7: Adjusted File import path.
Updated the import fromDrevOps\Installer\Utils\FiletoDrevOps\VortexInstaller\Utils\File..vortex/installer/tests/Unit/UnitTestCase.php (2)
5-5: Updated namespace in test fixture.
Changed fromDrevOps\Installer\Tests\UnittoDrevOps\VortexInstaller\Tests\Unitto reflect the new installer namespace.
9-9: Adjusted File utility import.
Import path updated to useDrevOps\VortexInstaller\Utils\Fileinstead of the old namespace..vortex/installer/src/Prompts/Handlers/PreserveDocsProject.php (2)
5-5: Updated namespace to VortexInstaller.
Changed namespace fromDrevOps\Installer\Prompts\HandlerstoDrevOps\VortexInstaller\Prompts\Handlers.
7-7: Adjusted File import path.
Updated import fromDrevOps\Installer\Utils\FiletoDrevOps\VortexInstaller\Utils\Filefor consistency..vortex/installer/src/Prompts/Handlers/DatabaseImage.php (2)
5-5: Updated namespace to VortexInstaller.
Namespace changed to match the renamed installer component.
7-8: Adjusted utility imports.
UpdatedEnvandFileimports toDrevOps\VortexInstaller\Utils\EnvandDrevOps\VortexInstaller\Utils\File..vortex/installer/src/Prompts/Handlers/PreserveDocsOnboarding.php (2)
5-5: Updated namespace to VortexInstaller.
Changed the namespace prefix toDrevOps\VortexInstaller\Prompts\Handlers.
7-7: Adjusted File import path.
Import updated fromDrevOps\Installer\Utils\FiletoDrevOps\VortexInstaller\Utils\File..vortex/installer/src/Prompts/Handlers/DependencyUpdatesProvider.php (2)
5-5: Namespace updated to VortexInstaller
The namespace declaration has been correctly changed fromDrevOps\Installer\Prompts\HandlerstoDrevOps\VortexInstaller\Prompts\Handlers.
7-7: Import adjusted for new namespace
TheFileutility import now correctly points toDrevOps\VortexInstaller\Utils\File..vortex/installer/tests/Unit/EnvTest.php (2)
5-5: Namespace updated in test
The test’s namespace has been updated toDrevOps\VortexInstaller\Tests\Unit, aligning with the new module prefix.
10-10: Env import adjusted for new namespace
TheEnvutility import now referencesDrevOps\VortexInstaller\Utils\Env..vortex/installer/tests/Unit/ValidatorTest.php (2)
5-5: Namespace updated in ValidatorTest
The namespace has been updated toDrevOps\VortexInstaller\Tests\Unitto reflect the refactoring.
9-9: Validator import adjusted for new namespace
TheValidatorutility import now points toDrevOps\VortexInstaller\Utils\Validator..vortex/installer/src/Prompts/Handlers/Webroot.php (1)
5-9: Namespace and imports updated to VortexInstaller
The namespace and imports forComposer,Env, andFilehave been correctly updated to theDrevOps\VortexInstaller\Utilsnamespace..vortex/installer/tests/Functional/InstallTest.php (2)
5-5: Functional test namespace updated
The test’s namespace is nowDrevOps\VortexInstaller\Tests\Functional, matching the new PSR-4 mapping.
7-41: Imports updated for new namespace prefix
Allusestatements have been correctly updated fromDrevOps\Installer\*toDrevOps\VortexInstaller\*across commands, handlers, and utilities..vortex/installer/src/Prompts/Handlers/Theme.php (2)
5-5: Namespace updated to VortexInstaller
Aligned the PSR-4 namespace with the newVortexInstallerprefix.
7-9: Imports updated to VortexInstaller\Utils
All utility class imports (Converter,Env,File) have been refactored to the new namespace..vortex/installer/src/Command/InstallCommand.php (3)
5-5: Namespace updated to VortexInstaller
PSR-4 namespace prefix has been updated correctly.
7-12: Use statements adjusted for new namespace
Imports ofPromptManager,Config,Downloader,Env,File, andTuinow point toDrevOps\VortexInstaller.
24-24: @Package annotation updated
Docblock package tag aligns with the newVortexInstaller\Commandnamespace..vortex/installer/tests/Functional/FunctionalTestCase.php (2)
5-5: Namespace updated to VortexInstaller
Functional tests now reference the correctDrevOps\VortexInstaller\Tests\Functionalnamespace.
10-15: Imports updated for VortexInstaller
ReplacedDrevOps\Installerimports withDrevOps\VortexInstallercounterparts forInstallCommand, traits, and utilities..vortex/installer/tests/Unit/PromptManagerTest.php (3)
5-5: Namespace updated to VortexInstaller
Unit tests now reside under the updatedDrevOps\VortexInstaller\Tests\Unitnamespace.
8-12: Core imports aligned to new namespace
Updated imports for theInternalhandler, upstream traits, and utility classes toDrevOps\VortexInstaller.
16-44: PromptManagerTest imports refactored
All prompt handler and utility imports have been migrated toDrevOps\VortexInstaller..vortex/installer/tests/Unit/FileTest.php (4)
5-5: Namespace updated to VortexInstaller
FileTest now uses theDrevOps\VortexInstaller\Tests\Unitnamespace.
7-7: Import of File utility updated
TheFileclass import has been updated toDrevOps\VortexInstaller\Utils\File.
72-74: Test for multiple empty lines remains valid
The input test for collapsing multiple empty lines after the pipe correctly verifies the behavior without modifications.
85-86: Whitespace-only empty lines test added
Including whitespace-only lines in the YAML literal block input enhances coverage of the collapse logic..vortex/installer/src/Prompts/PromptManager.php (1)
5-5:Details
✅ Verification successful
Namespace Declaration Updated
The new namespaceDrevOps\VortexInstaller\Promptsaligns with the refactoring. Please verify that your PSR-4 autoload mapping incomposer.jsonhas been updated from"DrevOps\\Installer\\":to"DrevOps\\VortexInstaller\\":accordingly.Run this to confirm:
🏁 Script executed:
rg '"DrevOps\\\\VortexInstaller"' composer.jsonLength of output: 49
I wasn’t able to find the PSR-4 mapping with the previous pattern—let’s search for any occurrences of “VortexInstaller” in your
composer.jsonto be sure:
🏁 Script executed:
#!/bin/bash rg 'VortexInstaller' -n composer.jsonLength of output: 37
To be sure we’re not missing any PSR-4 entries, let’s search for all “DrevOps” occurrences in your
composer.json:
🏁 Script executed:
#!/bin/bash grep -n '"DrevOps' composer.jsonLength of output: 34
Let’s inspect the PSR-4 autoload block to confirm the mapping update:
🏁 Script executed:
#!/bin/bash grep -n -C 2 '"psr-4"' composer.jsonLength of output: 38
Let’s locate every
composer.jsonin the repo and inspect its PSR-4 block:
🏁 Script executed:
#!/bin/bash find . -type f -name composer.json -exec grep -nC2 '"psr-4"' {} \;Length of output: 44376
PSR-4 Autoload Mapping for DrevOps\VortexInstaller Confirmed
The
composer.jsonalready contains the correct PSR-4 entry:• composer.json (lines 45–47)
"autoload": { "psr-4": { "DrevOps\\VortexInstaller\\": "src/" } }No further changes are needed.
| * Centralised place for providing prompts and their processing. | ||
| * | ||
| * @package DrevOps\Installer | ||
| * @package DrevOps\VortexInstaller |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Docblock @Package Tag
The @package annotation currently points to DrevOps\VortexInstaller. Consider including the sub-namespace for clarity: DrevOps\VortexInstaller\Prompts.
🤖 Prompt for AI Agents
In .vortex/installer/src/Prompts/PromptManager.php at line 54, update the
docblock @package annotation to include the sub-namespace by changing it from
DrevOps\VortexInstaller to DrevOps\VortexInstaller\Prompts for better clarity
and accuracy.
| $classes = []; | ||
| foreach ($handler_files as $file) { | ||
| $class = 'DrevOps\\Installer\\Prompts\\Handlers\\' . basename($file, '.php'); | ||
| $class = 'DrevOps\\VortexInstaller\\Prompts\\Handlers\\' . basename($file, '.php'); |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Use __NAMESPACE__ for Dynamic Handler Discovery
Hard-coding the namespace string can lead to drift if the namespace changes again. Refactor to use the __NAMESPACE__ constant:
- $class = 'DrevOps\\VortexInstaller\\Prompts\\Handlers\\' . basename($file, '.php');
+ $class = __NAMESPACE__ . '\\Handlers\\' . basename($file, '.php');📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| $class = 'DrevOps\\VortexInstaller\\Prompts\\Handlers\\' . basename($file, '.php'); | |
| $class = __NAMESPACE__ . '\\Handlers\\' . basename($file, '.php'); |
🤖 Prompt for AI Agents
In .vortex/installer/src/Prompts/PromptManager.php at line 663, the namespace
for the handler class is hard-coded as a string. Replace the hard-coded
namespace string with the __NAMESPACE__ constant concatenated with the handler
class name to ensure the namespace remains accurate if it changes. This involves
using __NAMESPACE__ instead of the literal
'DrevOps\\VortexInstaller\\Prompts\\Handlers\\' prefix.
| * PromptManager constructor. | ||
| * | ||
| * @param \DrevOps\Installer\Utils\Config $config | ||
| * @param \DrevOps\VortexInstaller\Utils\Config $config |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Simplify Docblock Param
You’ve imported Config; update the @param to use the short class name:
- * @param \DrevOps\VortexInstaller\Utils\Config $config
+ * @param Config $config🤖 Prompt for AI Agents
In .vortex/installer/src/Prompts/PromptManager.php at line 73, the docblock for
the parameter currently uses the fully qualified class name
\DrevOps\VortexInstaller\Utils\Config. Since Config is already imported, update
the @param annotation to use the short class name Config instead of the fully
qualified name to simplify the docblock.
| * Array of handlers. | ||
| * | ||
| * @var array<string, \DrevOps\Installer\Prompts\Handlers\HandlerInterface> | ||
| * @var array<string, \DrevOps\VortexInstaller\Prompts\Handlers\HandlerInterface> |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Simplify Docblock Type Hint
Since HandlerInterface is imported at the top, you can drop the fully-qualified name in the @var tag.
Apply:
- * @var array<string, \DrevOps\VortexInstaller\Prompts\Handlers\HandlerInterface>
+ * @var array<string, HandlerInterface>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| * @var array<string, \DrevOps\VortexInstaller\Prompts\Handlers\HandlerInterface> | |
| * @var array<string, HandlerInterface> |
🤖 Prompt for AI Agents
In .vortex/installer/src/Prompts/PromptManager.php at line 66, simplify the
docblock type hint by removing the fully-qualified namespace from the @var
annotation since HandlerInterface is already imported. Change the type hint from
array<string, \DrevOps\VortexInstaller\Prompts\Handlers\HandlerInterface> to
array<string, HandlerInterface>.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1721 +/- ##
========================================
Coverage 71.75% 71.75%
========================================
Files 77 77
Lines 4376 4376
========================================
Hits 3140 3140
Misses 1236 1236 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
1 participant
closes #1656
Summary by CodeRabbit