Refactor CI configuration and update test annotations for improved clarity

Author: calliostro

.gitattributes

@@ -0,0 +1,18 @@
+# Normalize line endings
+* text=auto
+
+# PHP files
+*.php text eol=lf diff=php
+
+# Config files
+*.json text eol=lf
+*.xml text eol=lf
+*.md text eol=lf
+
+# Exclude from releases
+.gitattributes export-ignore
+.gitignore export-ignore
+tests/ export-ignore
+phpunit.xml* export-ignore
+.phpunit.* export-ignore
+codecov.yml export-ignore

.github/workflows/ci.yml

@@ -78,11 +78,8 @@ jobs:
           composer config minimum-stability ${{ matrix.stability }}
           composer config prefer-stable true
 
-      - name: Remove composer.lock
-        run: rm -f composer.lock
-
       - name: Install dependencies
-        run: composer update --prefer-dist --no-interaction --no-progress
+        run: composer install --prefer-dist --no-interaction --no-progress
 
       - name: Validate composer.json and composer.lock
         run: composer validate --strict

DEVELOPMENT.md

@@ -44,8 +44,8 @@ Integration tests are **separated from the CI pipeline** to prevent:
 
 ### Test Strategy
 
-- **Unit Tests (101)**: Fast, reliable, no external dependencies → **CI default**
-- **Integration Tests (31)**: Real API calls, rate-limited → **Manual execution**  
+- **Unit Tests**: Fast, reliable, no external dependencies → **CI default**
+- **Integration Tests**: Real API calls, rate-limited → **Manual execution**
 - **Total Coverage**: 100% lines, methods, and classes covered
 
 ### GitHub Secrets Required

UPGRADE.md

@@ -36,7 +36,7 @@ $search = $discogs->search('Billie Eilish', 'artist');
 - **Developer Experience**: ~750 lines of focused code with comprehensive type safety
 - **Type Safety**: Automatic parameter validation and conversion
 
-## � Migration Steps
+## 🔄 Migration Steps
 
 ### Step 1: Update Dependencies
 
@@ -89,7 +89,7 @@ Parameters follow the order defined in the [service configuration](resources/ser
 
 **💡 Tip**: Use `null` for optional parameters you want to skip.
 
-## � Migration Examples
+## 📚 Migration Examples
 
 ### Database Methods
 

phpunit.xml.dist

@@ -29,5 +29,12 @@
   <php>
     <ini name="error_reporting" value="E_ALL"/>
     <ini name="memory_limit" value="-1"/>
+    <!-- Discogs API credentials for integration tests -->
+    <!-- Set these environment variables before running integration tests -->
+    <env name="DISCOGS_CONSUMER_KEY" value=""/>
+    <env name="DISCOGS_CONSUMER_SECRET" value=""/>
+    <env name="DISCOGS_PERSONAL_ACCESS_TOKEN" value=""/>
+    <env name="DISCOGS_OAUTH_TOKEN" value=""/>
+    <env name="DISCOGS_OAUTH_TOKEN_SECRET" value=""/>
   </php>
 </phpunit>

tests/Integration/AuthenticatedIntegrationTest.php

@@ -9,9 +9,6 @@
 
 /**
  * Integration tests that require authentication credentials
- *
- * @group integration
- * @group authenticated
  */
 final class AuthenticatedIntegrationTest extends IntegrationTestCase
 {

tests/Integration/ClientWorkflowTest.php

@@ -11,16 +11,16 @@
 use GuzzleHttp\Handler\MockHandler;
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\Response;
+use PHPUnit\Framework\Attributes\CoversClass;
 use ReflectionClass;
 use ReflectionException;
 use RuntimeException;
 
 /**
  * Integration tests for the complete client workflow
- *
- * @covers \Calliostro\Discogs\DiscogsClientFactory
- * @covers \Calliostro\Discogs\DiscogsClient
  */
+#[CoversClass(DiscogsClientFactory::class)]
+#[CoversClass(DiscogsClient::class)]
 final class ClientWorkflowTest extends IntegrationTestCase
 {
     /**

tests/Unit/DiscogsClientFactoryTest.php

@@ -13,11 +13,11 @@
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Middleware;
 use GuzzleHttp\Psr7\Response;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\UsesClass;
 
-/**
- * @covers \Calliostro\Discogs\DiscogsClientFactory
- * @uses   \Calliostro\Discogs\DiscogsClient
- */
+#[CoversClass(DiscogsClientFactory::class)]
+#[UsesClass(DiscogsClient::class)]
 final class DiscogsClientFactoryTest extends UnitTestCase
 {
     /**

tests/Unit/DiscogsClientTest.php

@@ -16,16 +16,15 @@
 use GuzzleHttp\Psr7\Request;
 use GuzzleHttp\Psr7\Response;
 use InvalidArgumentException;
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\MockObject\MockObject;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\StreamInterface;
 use ReflectionClass;
 use ReflectionException;
 use RuntimeException;
 
-/**
- * @covers \Calliostro\Discogs\DiscogsClient
- */
+#[CoversClass(DiscogsClient::class)]
 final class DiscogsClientTest extends UnitTestCase
 {
     private DiscogsClient $client;

tests/Unit/OAuthHelperTest.php

@@ -12,11 +12,10 @@
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\Request;
 use GuzzleHttp\Psr7\Response;
+use PHPUnit\Framework\Attributes\CoversClass;
 use RuntimeException;
 
-/**
- * @covers \Calliostro\Discogs\OAuthHelper
- */
+#[CoversClass(OAuthHelper::class)]
 final class OAuthHelperTest extends UnitTestCase
 {
     public function testGetAuthorizationUrl(): void