Skip to content

Refresh Skin Designer device database#4862

Closed
github-actions[bot] wants to merge 1 commit into
masterfrom
automation/skin-designer-device-db
Closed

Refresh Skin Designer device database#4862
github-actions[bot] wants to merge 1 commit into
masterfrom
automation/skin-designer-device-db

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 4, 2026

Summary

  • Regenerates scripts/skindesigner/common/src/main/resources/devices.json
    by scraping a small batch of the latest devices from GSMArena and
    merging into the existing catalog.

Created automatically by the Skin Designer device DB workflow.

@github-actions github-actions Bot added automation dependencies Pull requests that update a dependency file labels May 4, 2026
@github-actions github-actions Bot force-pushed the automation/skin-designer-device-db branch from 29e3a9c to 645de65 Compare May 4, 2026 18:44
@shai-almog shai-almog closed this May 4, 2026
@shai-almog shai-almog reopened this May 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented May 4, 2026

Cloudflare Preview

@shai-almog shai-almog closed this May 5, 2026
shai-almog added a commit that referenced this pull request May 5, 2026
The scheduled trickle scrape has been hitting a 404 since #4758 landed
(latest-mobiles.php3 doesn't exist on GSMArena), so every 6h run pulled
zero phones and only opened PRs because the JSON envelope churned
between runs (e.g. #4862). Replace the trickle path with a weekly run
of the verified per-brand walk, and make the script skip writing the
file when no device records actually change so envelope drift can't
spam PRs again.

- Workflow: weekly Mon 03:00 UTC, single brands scrape, 75m timeout
  (cold cache). Drop the mode dispatch input and trickle/full split.
- Script: drop dead --mode latest path (walk_latest, RE_LATEST_LINK,
  --mode flag). Add _devices_changed() so the file is only rewritten
  when actual records added/removed/modified, ignoring envelope diffs.
- README: document the new weekly cadence.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants