[2.x] Detect Claude via AI_AGENT prefix

[jackbayliss]

I was playing around with this locally, just to see what I could do with it.

Used the code snippet in the readme, but noticed it wasn't detecting Claude as a known agent.

Code snippet

<?php

namespace App\Console\Commands;

use Illuminate\Console\Attributes\Description;
use Illuminate\Console\Attributes\Signature;
use Illuminate\Console\Command;
use Laravel\AgentDetector\AgentDetector;
use Laravel\AgentDetector\KnownAgent;

#[Signature('app:example')]
#[Description('Command description')]
class ExampleCommand extends Command
{
    /**
     * Execute the console command.
     */
    public function handle()
    {
        $this->info('Hello World!');
        $result = AgentDetector::detect();

        if ($result->isAgent) {
            $this->info("Running inside: {$result->name}");
        }

        if ($result->knownAgent() === KnownAgent::Claude) {
            $this->info("Hello from Claude!");
        }
    }
}

I was expecting to see 'Hello from Claude':

With the change, its now detected properly:

I've put the logic in the match as it felt nicer than having a bunch of return points around name matching.

AI_AGENT is checked first afaik, which is why CLAUDECODE by itself doesnt trigger the right name, this could happen for other agents too.

Maybe self::fromKnownEnvVars() should be checked first? But I'm not sure, so went with this surgical approach for now :)

[github-actions]

Thanks for submitting a PR!

Note that draft PRs are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

[pushpak1300]

I think we can move AI_AGENT check at the end.

[jackbayliss]

@pushpak1300 I did think the same, but prioritizes CURSOR_AGENT over CLAUDECODE fails is we do that. Perhaps you can see a better way though? Thanks for the reply 👍🏻

[pushpak1300]

Thanks.