style: fix phpcs errors introduced by #416 refactor

- WorktreeContextInjector.php:590 — split the canonical-top-level array
  declaration across multiple lines per WordPress.Arrays.ArrayDeclaration
  Spacing.AssociativeArrayFound.
- WorktreeContextInjector.php:596 — flip the underscore-position guard to
  Yoda style (`strlen( $key ) - 1 === $underscore`).
- WorktreeContextInjector.php:1557 — flip the handle equality check to
  Yoda style (the casted lookup is now on the left).

No behavior change. All session-attribution smoke tests still pass.

Refs #416

Author: chubes4

inc/Workspace/WorktreeContextInjector.php

@@ -587,13 +587,16 @@ private static function migrate_legacy_origin_session( array $session ): array {
 
 		// legacy migration only: split top-level `<runtime>_<subkey>` keys into
 		// the runtime-keyed envelope. Skip canonical envelope keys.
-		$canonical_top_level = array( 'primary_id' => true, 'ids' => true );
+		$canonical_top_level = array(
+			'primary_id' => true,
+			'ids'        => true,
+		);
 		foreach ( $session as $key => $value ) {
 			if ( ! is_string( $key ) || isset( $canonical_top_level[ $key ] ) ) {
 				continue;
 			}
 			$underscore = strpos( $key, '_' );
-			if ( false === $underscore || 0 === $underscore || $underscore === strlen( $key ) - 1 ) {
+			if ( false === $underscore || 0 === $underscore || strlen( $key ) - 1 === $underscore ) {
 				continue;
 			}
 			$runtime_id = substr( $key, 0, $underscore );
@@ -1554,7 +1557,7 @@ private static function get_inventory_metadata( string $handle ): ?array {
 		}
 
 		foreach ( $repository->list() as $row ) {
-			if ( $handle === (string) ( $row['handle'] ?? '' ) && is_array( $row['metadata'] ?? null ) ) {
+			if ( (string) ( $row['handle'] ?? '' ) === $handle && is_array( $row['metadata'] ?? null ) ) {
 				return (array) $row['metadata'];
 			}
 		}