Compare commits

...

2 commits

Author SHA1 Message Date
dlawler489
517574720e Merge watch check fix
All checks were successful
build-and-push / docker (push) Successful in 15s
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-21 10:48:45 +10:00
dlawler489
ca0d0d52f5 Fix a watch check stopping after one model
withFiles was declared inside the try block but read by the wait after it,
so every check threw ReferenceError once it had imported a single model.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-21 10:48:45 +10:00

View file

@ -184,10 +184,11 @@ async function importFound(watch, collection) {
const { rows: [live] } = await db.query('SELECT enabled FROM watched_collections WHERE id = $1', [watch.id]); const { rows: [live] } = await db.query('SELECT enabled FROM watched_collections WHERE id = $1', [watch.id]);
if (!live || !live.enabled) break; if (!live || !live.enabled) break;
const url = makerworld.designUrl(design); const url = makerworld.designUrl(design);
// While MakerWorld is waiting for you to prove you're not a robot, the
// check still brings in the models — their files wait for you. Decided out
// here because the wait after each import depends on it.
const withFiles = Boolean(watch.fetch_files) && !downloadBlock.isBlocked('makerworld');
try { try {
// While MakerWorld is waiting for you to prove you're not a robot, the
// check still brings in the models — their files wait for you.
const withFiles = Boolean(watch.fetch_files) && !downloadBlock.isBlocked('makerworld');
const result = await deps.importFromUrl(url, { const result = await deps.importFromUrl(url, {
projects: watch.project_id ? [watch.project_id] : [], projects: watch.project_id ? [watch.project_id] : [],
withFiles, // off by default: MakerWorld refuses quick bursts withFiles, // off by default: MakerWorld refuses quick bursts