Resolve compatibility issues with latest Valet (#44)

* fix sanitization of site_name

* add ValetConfig

* add Config facade

* bind config to ValetConfig implementations

* replace Valet::domain with Config

* remove unused domain methods

Author: aaemnnosttv

src/Config.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace WP_CLI_Valet;
+
+/**
+ * @method static get(string $key)
+ */
+class Config extends Facade
+{
+    public static function getContainerKey()
+    {
+        return 'config';
+    }
+}

src/Process/FakeValet.php

@@ -4,17 +4,6 @@
 
 class FakeValet implements ValetInterface
 {
-
-    /**
-     * Get the local domain served by Valet.
-     *
-     * @return string
-     */
-    public function domain()
-    {
-        return 'dev';
-    }
-
     /**
      * Secure the installation with a self-signed TLS certificate.
      *

src/Process/SystemValet.php

@@ -6,16 +6,6 @@ class SystemValet extends ShellCommand implements ValetInterface
 {
     protected $command = 'valet';
 
-    /**
-     * Get the local domain served by Valet.
-     *
-     * @return string
-     */
-    public function domain()
-    {
-        return trim($this->run('domain')->stdout);
-    }
-
     /**
      * Secure the installation with a self-signed TLS certificate.
      *

src/Process/ValetInterface.php

@@ -4,13 +4,6 @@
 
 interface ValetInterface
 {
-    /**
-     * Get the local domain served by Valet.
-     *
-     * @return string
-     */
-    public function domain();
-
     /**
      * Secure the installation with a self-signed TLS certificate.
      *

src/Props.php

@@ -33,8 +33,8 @@ public function __construct(array $positional, array $options)
      */
     public function populate()
     {
-        $this->site_name = preg_replace('/^a-zA-Z/', '-', $this->positional[0]);
-        $this->domain    = sprintf('%s.%s', $this->site_name, Valet::domain());
+        $this->site_name = preg_replace('/[^a-zA-Z0-9\.]/', '-', $this->positional[0]);
+        $this->domain    = sprintf('%s.%s', $this->site_name, Config::get('tld') ?: Config::get('domain'));
     }
 
     /**

src/ValetCommand.php

@@ -48,6 +48,11 @@ public static function boot()
         $container->singleton('valet', getenv('BEHAT_RUN') ? FakeValet::class : SystemValet::class);
         $container->singleton('wp', SystemWp::class);
         $container->singleton('composer', SystemComposer::class);
+        $container->singleton('config', function () {
+            return getenv('BEHAT_RUN')
+                ? new ValetConfig(['tld' => 'dev'])
+                : ValetConfig::loadSystem();
+        });
 
         $container->bind('wp-installer', WordPressInstaller::class);
         $container->bind('bedrock-installer', BedrockInstaller::class);

src/ValetConfig.php

@@ -0,0 +1,39 @@
+<?php
+
+namespace WP_CLI_Valet;
+
+class ValetConfig
+{
+    protected $data;
+
+    /**
+     * ValetConfig constructor.
+     *
+     * @param $data
+     */
+    public function __construct($data)
+    {
+        $this->data = $data;
+    }
+
+    /**
+     * @throws \Exception
+     */
+    public static function loadSystem()
+    {
+        $config_path = file_exists("{$_SERVER['HOME']}/.config/valet/config.json")
+            ? "{$_SERVER['HOME']}/.config/valet/config.json"
+            : "{$_SERVER['HOME']}/.valet/config.json";
+
+        if (! file_exists($config_path)) {
+            throw new \Exception('Valet configuration file not found.');
+        }
+
+        return new static(json_decode(file_get_contents($config_path), true));
+    }
+
+    public function get($key)
+    {
+        return isset($this->data[$key]) ? $this->data[$key] : null;
+    }
+}