cleaniquecoders / packager

src/Hook/MakeHookCommand.php

<?php

namespace CleaniqueCoders\Console\Hook;

use CleaniqueCoders\Console\Commander;
use Illuminate\Support\Str;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class MakeHookCommand extends Commander
{
    /**
     * Configure the command options.
     */
    protected function configure()
    {
        $this
            ->setName('hook')
            ->setDescription('Hook package to a Laravel project locally')
            ->addArgument('name', InputArgument::REQUIRED)
            ->addArgument('to', InputArgument::REQUIRED);
    }

    /**
     * Execute the command.
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $name      = $input->getArgument('name');
        $pathOrUrl = $input->getArgument('to');

        $status = $this->composerLink(Str::snake($name), $pathOrUrl);

It seems like $name can also be of type string[]; however, parameter $value of Illuminate\Support\Str::snake() does only seem to accept string, maybe add an additional type check?

It seems like $pathOrUrl can also be of type string[]; however, parameter $pathOrUrl of CleaniqueCoders\Console\Commander::composerLink() does only seem to accept string, maybe add an additional type check?

        if ($status) {

The condition $status is always true.

            $output->writeln('<info>Packaged linked.</info>');
        } else {
            $output->writeln('<comment>Unable to link the package.</comment>');
        }
    }
}