refactor(composer): cleanup and normalize composer.json

[nortej-hash]

• Did you create an Apache Jira ticket? (Request account here, not required for trivial changes)
• If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
• Did you squash your changes to a single commit? (not required, but preferred)
• Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change"?
• If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

[Jens-G]

Code review

Found 3 issues:

1. "readme": "README.md" is removed. The PR is presented as a formatting cleanup, but this is a content change that neither the title nor the description mentions. The practical effect is nil -- Packagist auto-detects README.md for GitHub-hosted packages -- but the key was added deliberately in 384647d29 ("Version 0.12.0").

thrift/composer.json

Lines 7 to 9 in b8da62a

	"license": "Apache-2.0",
	"readme": "README.md",
	"authors": [

2. "minimum-stability": "stable" is removed. The same commit set this deliberately, changing it from "dev" to "stable" as part of making the PHP package release-ready. Composer's default is already stable, so behaviour does not change, but the explicit declaration goes away without a note.

thrift/composer.json

Lines 58 to 60 in b8da62a

	},
	"minimum-stability": "stable",
	"extra": {

3. authors[0].homepage is removed. Present since the file was first added; it drops the author-block link Packagist renders. Cosmetic, but likewise undisclosed.

thrift/composer.json

Lines 12 to 14 in b8da62a

	"email": "dev@thrift.apache.org",
	"homepage": "http://thrift.apache.org"
	}

Each is harmless on its own. The ask is to either restore them or name them in the PR description, so that a reformat PR reviews as a reformat. The require/require-dev sets, autoload paths and archive.exclude are unchanged, and the 4-to-2-space reindent brings the file in line with .editorconfig's [*.json] indent_size = 2.

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

[Jens-G]

@sveneld Could you have a look from your side and merge if all is fine?

[sveneld]

I’ll look at it over the weekend.