php/archived-pie
1
0
Fork 0
mirror of https://github.com/php/pie.git synced 2026-03-23 23:12:17 +01:00
Code Issues Packages Projects Releases Wiki Activity

434: improved messaging and flow for prompts

Browse Source
This commit is contained in:
James Titcumb
2025-12-04 17:12:33 +00:00
parent 4a85df4657
commit ae10e7fce2
1 changed files with 30 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
src/SelfManage/BuildTools/CheckAllBuildTools.php
View File

@@ -7,6 +7,8 @@ namespace Php\Pie\SelfManage\BuildTools;
use Composer\IO\IOInterface;
use function array_unique;
use function array_values;
use function count;
use function implode;
/** @internal This is not public API for PIE, so should not be depended upon unless you accept the risk of BC breaks */
@@ -76,9 +78,10 @@ class CheckAllBuildTools
public function check(IOInterface $io, bool $forceInstall): void
{
$io->write('<info>Checking if all build tools are installed.</info>', verbosity: IOInterface::VERBOSE);
$packageManager = PackageManager::detect();
$missingTools = [];
/** @var list<string> $packagesToInstall */
$packagesToInstall = [];
$missingTools = [];
$packageManager = PackageManager::detect();
$allFound = true;
foreach ($this->buildTools as $buildTool) {
@@ -89,10 +92,15 @@ class CheckAllBuildTools
$allFound = false;
$missingTools[] = $buildTool->tool;
$packageName = $buildTool->packageNameFor($packageManager);
if ($packageManager === null) {
continue;
}
$packageName = $buildTool->packageNameFor($packageManager);
if ($packageName === null) {
$io->writeError('<warning>Could not find package name for build tool ' . $buildTool->tool . '.</warning>');
$io->writeError('<warning>Could not find package name for build tool ' . $buildTool->tool . '.</warning>', verbosity: IOInterface::VERBOSE);
continue;
}
@@ -107,15 +115,27 @@ class CheckAllBuildTools
$io->write('<comment>The following build tools are missing: ' . implode(', ', $missingTools) . '</comment>');
if (! $io->isInteractive() && ! $forceInstall) {
$io->writeError('<error>You are not running in interactive mode, and --force was not specified. You may need to install the following build tools: ' . implode(' ', $packagesToInstall) . '</error>');
if ($packageManager === null) {
$io->write('<warning>Could not find a package manager to install the missing build tools.</warning>');
return;
}
$packagesToInstall = array_unique($packagesToInstall);
if (! count($packagesToInstall)) {
$io->write('<warning>Could not determine packages to install.</warning>');
$io->write('The following command will be run: ' . implode(' ', $packageManager->installCommand($packagesToInstall)), verbosity: IOInterface::VERBOSE);
return;
}
$proposedInstallCommand = implode(' ', $packageManager->installCommand(array_values(array_unique($packagesToInstall))));
if (! $io->isInteractive() && ! $forceInstall) {
$io->writeError('<warning>You are not running in interactive mode. You may need to run: ' . $proposedInstallCommand . '</warning>');
return;
}
$io->write('The following command will be run: ' . $proposedInstallCommand, verbosity: IOInterface::VERY_VERBOSE);
if ($io->isInteractive() && ! $forceInstall) {
if (! $io->askConfirmation('<question>Would you like to install them now?</question>', false)) {
@@ -125,7 +145,7 @@ class CheckAllBuildTools
}
}
$packageManager->install($packagesToInstall);
$io->write('<info>Build tools installed.</info>');
$packageManager->install(array_values(array_unique($packagesToInstall)));
$io->write('<info>Missing build tools have been installed.</info>');
}
}