troy: Skill for Cursor

Trailing commas

Source of truth: .github/copilot-instructions.md — trailing commas at the end of multiline object/array properties and function arguments when the language supports it.

PHP 7.4

Put a trailing comma on multiline:

• Function and method calls (apply_filters(, sprintf(, Escape::css_content()
• array( … ) and [ … ] literals

Do not put a trailing comma on:

• Function, method, closure, or fn() parameter lists in PHP 7.4 packages (PHP 8.0 syntax)
• Closure use ( … ) lists (PHP 8.0)
• if ( / elseif ( / for ( / foreach ( / while ( / switch ( / catch ( conditions
• Single-line calls or arrays

src/troy-server/** is PHP 8.4+. Put a trailing comma on multiline parameter lists there, matching existing constructors. PHP 7.4 packages must not.

JavaScript

Same rule for multiline calls, arrays, and objects. Parameter lists may have trailing commas (ES2017). Do not edit *.min.js.

Shapes

Comma after the last item of a multiline call, array, or object. A docblock after ( does not make it single-line.

foo( $a, $b );           // no — one line
foo(
	$a,
	$b,                  // yes
);
bar(
	baz( $x ),           // yes — bar is multiline
);
bar(
	baz(
		$x,              // yes — both
	),
);
[
	'a' => 1,
	'b' => 2,            // yes
];
function f( $a, $b ) {}  // no — one line; PHP 7.4 packages: never on params
function f(             // troy-server (PHP 8.4) multiline params: yes
	$a,
	$b,
) {}
use ( $c )               // no — closure use (PHP 8.0)
if ( $a && $b ) {}       // no — condition

fn( a, b );              // no — one line
fn(
	a,
	b,                   // yes
);
{ a: 1, b: 2 };          // no — one line
{
	a: 1,
	b: 2,                // yes
}
function f( a, b ) {}    // yes — JS parameters may

Finder

PHP only. Same command on Windows (PowerShell or cmd), macOS, and Linux. PHP 7.4+ on PATH. Reports; does not rewrite.

php .cursor/skills/trailing-comma/scripts/find-missing.php path/to/file.php
php .cursor/skills/trailing-comma/scripts/find-missing.php src