Merge pull request #26 from yoavf/fix/caching

Remove browser caching to prevent compatibility issues with new Playwright versions

Author: yoavf

CHANGELOG.md

@@ -1,30 +1,15 @@
 # Changelog
 
-## Unreleased
-
-### Added
-- Automatic Playwright browser caching between runs
-- Smart cache key generation based on Playwright version, OS, architecture, and browser selection
-- Automatic cache restoration and saving with fallback to fresh installation
-
-### Changed
-- Browser installation is now handled by a dedicated caching module
-- Improved error handling and logging for browser setup
-
 ## [1.3.0-beta](https://github.com/yoavf/auto-pr-screenshots-action/compare/v1.2.0-beta...v1.3.0-beta) (2025-08-19)
 
-
 ### Features
 
-* add browser caching ([d288d18](https://github.com/yoavf/auto-pr-screenshots-action/commit/d288d1822a85f2e4f3881dbbcb72795cb9d9814f))
 * add release-please workflow ([8826e1c](https://github.com/yoavf/auto-pr-screenshots-action/commit/8826e1c0fc6919d93850eb687fcf214dbec0f29c))
 
-
 ### Bug Fixes
 
 * invalid yaml file for release-please workflow ([bed9ef2](https://github.com/yoavf/auto-pr-screenshots-action/commit/bed9ef2f91c5677d11a00ed0368e16645fb3ed4e))
 * invalid yaml file for workflow ([0cf0c26](https://github.com/yoavf/auto-pr-screenshots-action/commit/0cf0c268050ae8611417e6be8cf1db20bdd1c6f2))
-
 ## 1.1.0
 
 - Improve the layout of the comment posted on the PR

README.md

@@ -29,22 +29,22 @@ jobs:
     permissions:
       contents: write      # Required for pushing screenshots
       pull-requests: write # Required for posting comments
-
+    
     steps:
       - uses: actions/checkout@v4
-
+      
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
           node-version: '20'
-
+      
       # Start your app (example)
       - run: npm install
       - run: npm run dev &
-
+      
       # Wait for your app to be ready
       - run: npx wait-on http://localhost:3000
-
+      
       - name: Take screenshots
         uses: yoavf/auto-pr-screenshots@v1
         with:
@@ -76,14 +76,14 @@ screenshots:
       width: 1440
       height: 900
     wait_for: '[data-testid="hero-section"]'
-
+    
   - name: home-mobile
     url: http://localhost:3000
     viewport:
       width: 390
       height: 844
       deviceScaleFactor: 3
-
+    
   - name: dashboard
     url: http://localhost:3000/dashboard
     viewport:
@@ -154,12 +154,6 @@ Available step types:
 - `wait`: Wait for milliseconds
 - `wait_for`: Wait for element
 
-## Performance Features
-
-### Browser Caching
-
-This action automatically caches Playwright browser binaries between runs. The caching works transparently without any configuration needed from users.
-
 ## Examples
 
 ### Next.js App
@@ -207,13 +201,13 @@ screenshots:
     viewport:
       width: 1440
       height: 900
-
+  
   - name: about
     url: http://localhost:3000/about
     viewport:
       width: 1440
       height: 900
-
+  
   - name: contact
     url: http://localhost:3000/contact
     viewport:
@@ -267,7 +261,7 @@ steps:
       selector: 'input[name="username"]'
       text: testuser
   - fill:
-      selector: 'input[name="password"]'
+      selector: 'input[name="password"]'  
       text: testpass
   - click: 'button[type="submit"]'
   - wait_for: '[data-testid="user-dashboard"]'

action.yml

@@ -11,32 +11,32 @@ inputs:
     description: 'URL of the frontend to capture screenshots from'
     required: false
     default: ''
-
+  
   config-file:
     description: 'Path to configuration file (optional if URL is provided)'
     required: false
     default: '.github/screenshots.config.yml'
-
+  
   browsers:
     description: 'Playwright browsers to use (chromium, firefox, webkit)'
     required: false
     default: 'chromium'
-
+  
   skip-comment:
     description: 'Skip posting comment to PR'
     required: false
     default: 'false'
-
+  
   fail-on-error:
     description: 'Fail the action if screenshot capture fails'
     required: false
     default: 'true'
-
+  
   branch:
     description: 'Branch name for storing screenshots'
     required: false
     default: 'gh-screenshots'
-
+  
   github-token:
     description: 'GitHub token for posting comments and pushing screenshots'
     required: true