fix(cli): adopt core records canonical hashes and flags local edits
This commit is contained in:
@@ -89,7 +89,7 @@ final class AdoptCommand extends Command
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
$coreAdopted = $this->adoptCore($parsed['options'], $root, $lock, $dryRun);
|
[$coreAdopted, $coreHasLocal] = $this->adoptCore($parsed['options'], $root, $lock, $dryRun);
|
||||||
|
|
||||||
if (!$dryRun) {
|
if (!$dryRun) {
|
||||||
$lock->save($root);
|
$lock->save($root);
|
||||||
@@ -100,7 +100,8 @@ final class AdoptCommand extends Command
|
|||||||
$this->console->ok('adoptado ' . $id . $mark);
|
$this->console->ok('adoptado ' . $id . $mark);
|
||||||
}
|
}
|
||||||
if ($coreAdopted > 0) {
|
if ($coreAdopted > 0) {
|
||||||
$this->console->ok("adoptado el core ({$coreAdopted} archivos)");
|
$mark = $coreHasLocal ? ' (con ediciones locales)' : '';
|
||||||
|
$this->console->ok("adoptado el core ({$coreAdopted} archivos){$mark}");
|
||||||
}
|
}
|
||||||
if ($adopted === [] && $coreAdopted === 0) {
|
if ($adopted === [] && $coreAdopted === 0) {
|
||||||
$this->console->out('No se encontró nada que adoptar.');
|
$this->console->out('No se encontró nada que adoptar.');
|
||||||
@@ -111,33 +112,42 @@ final class AdoptCommand extends Command
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Registra el core existente: sólo los archivos que también existen en el
|
* Registra el core existente: sólo los archivos que también existen en el
|
||||||
* core de referencia, con su hash actual. No toca archivos.
|
* core de referencia, con su hash CANÓNICO. Si la copia local difiere,
|
||||||
|
* el archivo queda marcado como edición local y `update core` preguntará
|
||||||
|
* antes de pisarla (mismo contrato que `adoptComponent`).
|
||||||
*
|
*
|
||||||
* @param array<string, string|bool> $options
|
* @param array<string, string|bool> $options
|
||||||
|
* @return array{0:int,1:bool}
|
||||||
*/
|
*/
|
||||||
private function adoptCore(array $options, string $root, $lock, bool $dryRun): int
|
private function adoptCore(array $options, string $root, $lock, bool $dryRun): array
|
||||||
{
|
{
|
||||||
if ($lock->core() !== null) {
|
if ($lock->core() !== null) {
|
||||||
return 0;
|
return [0, false];
|
||||||
}
|
}
|
||||||
|
|
||||||
try {
|
try {
|
||||||
$provider = $this->coreProvider($options);
|
$provider = $this->coreProvider($options);
|
||||||
} catch (ToolboxException $e) {
|
} catch (ToolboxException $e) {
|
||||||
$this->console->warn('no se pudo resolver el core para adoptar: ' . $e->getMessage());
|
$this->console->warn('no se pudo resolver el core para adoptar: ' . $e->getMessage());
|
||||||
return 0;
|
return [0, false];
|
||||||
}
|
}
|
||||||
|
|
||||||
$files = [];
|
$files = [];
|
||||||
|
$hasLocal = false;
|
||||||
foreach ($provider->files() as $relative) {
|
foreach ($provider->files() as $relative) {
|
||||||
$absolute = $root . '/' . $relative;
|
$absolute = $root . '/' . $relative;
|
||||||
if (is_file($absolute)) {
|
if (!is_file($absolute)) {
|
||||||
$files[$relative] = Drift::hash($absolute);
|
continue;
|
||||||
|
}
|
||||||
|
$canonical = $provider->read($relative);
|
||||||
|
$files[$relative] = 'sha256:' . hash('sha256', $canonical);
|
||||||
|
if (file_get_contents($absolute) !== $canonical) {
|
||||||
|
$hasLocal = true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($files === []) {
|
if ($files === []) {
|
||||||
return 0;
|
return [0, false];
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!$dryRun) {
|
if (!$dryRun) {
|
||||||
@@ -145,9 +155,10 @@ final class AdoptCommand extends Command
|
|||||||
'version' => $provider->version(),
|
'version' => $provider->version(),
|
||||||
'commit' => $provider->commit(),
|
'commit' => $provider->commit(),
|
||||||
'files' => $files,
|
'files' => $files,
|
||||||
|
'origin' => $hasLocal ? 'local' : 'adopted',
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
return count($files);
|
return [count($files), $hasLocal];
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -121,3 +121,31 @@ Harness::it('adopt registra el core existente con sus hashes', function () {
|
|||||||
rrmdir($proj);
|
rrmdir($proj);
|
||||||
rrmdir($core);
|
rrmdir($core);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
Harness::it('adopt del core registra hash canónico y marca ediciones locales', function () {
|
||||||
|
$repo = $GLOBALS['g9_repo'];
|
||||||
|
$core = g7_core('0.1.0');
|
||||||
|
$proj = tmpdir('proj');
|
||||||
|
|
||||||
|
// Copia idéntica de Router.php + config.php customizado (proyecto heredado).
|
||||||
|
write_file($proj . '/src/Libs/Router.php', (string) file_get_contents($core . '/src/Libs/Router.php'));
|
||||||
|
write_file($proj . '/config.php', "<?php\n// credenciales del proyecto\n");
|
||||||
|
|
||||||
|
(new AdoptCommand(new Console()))->run(['--root=' . $proj, '--toolbox=' . $repo, '--core-dir=' . $core]);
|
||||||
|
|
||||||
|
$lock = Lock::load($proj);
|
||||||
|
$coreHashRouter = $lock->core()['files']['src/Libs/Router.php'];
|
||||||
|
$coreHashConfig = $lock->core()['files']['config.php'];
|
||||||
|
|
||||||
|
$canonicalRouter = (string) file_get_contents($core . '/src/Libs/Router.php');
|
||||||
|
$canonicalConfig = (string) file_get_contents($core . '/config.php');
|
||||||
|
|
||||||
|
eq('sha256:' . hash('sha256', $canonicalRouter), $coreHashRouter, 'hash de Router.php = canónico');
|
||||||
|
eq('sha256:' . hash('sha256', $canonicalConfig), $coreHashConfig, 'hash de config.php = canónico (aunque el local difiera)');
|
||||||
|
eq('local', $lock->core()['origin'] ?? null, 'origin = local porque hay ediciones locales');
|
||||||
|
|
||||||
|
ok(file_get_contents($proj . '/config.php') !== $canonicalConfig, 'la copia local customizada no se tocó');
|
||||||
|
|
||||||
|
rrmdir($proj);
|
||||||
|
rrmdir($core);
|
||||||
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user