mtvbrianking / laravel-airtel-money

src/Traits/CommandUtils.php

<?php

namespace Bmatovu\AirtelMoney\Traits;

trait CommandUtils
{
    protected function persistConfig(string $configKey, ?string $value): void
    {
        $this->laravel->make('config')->set([$configKey => $value]);

        if ($this->option('no-write')) {

It seems like option() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return;
        }

        $envPath = app()->basePath('.env');

The method basePath() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        $envKey = strtoupper(preg_replace('/[^A-Za-z0-9]+/', '_', $configKey));

        $pattern = '/^'.preg_quote($envKey, '/').'=["\']?.*/m';

        $contents = file_get_contents($envPath);

        if (preg_match($pattern, $contents)) {
            $contents = preg_replace($pattern, "{$envKey}=\"{$value}\"", $contents);
        } else {
            $contents .= PHP_EOL."{$envKey}=\"{$value}\"";
        }

        file_put_contents($envPath, $contents);
    }

    protected function writeConfig(
        string $key,
        ?string $value = null,
        string $config = 'airtel-money'
    ): ?string {
        $configKey = "{$config}.{$key}";

        $oldValue = $value ?? $this->laravel->make('config')->get($configKey);

        $newValue = $this->ask($key, $oldValue);

It seems like ask() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->persistConfig($configKey, $newValue);

        return $newValue;
    }
}