dlawler489/printhive
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix a watch check stopping after one model

Browse source 
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>
This commit is contained in:
dlawler489 2026-09-21 10:48:45 +10:00
parent 4956eabbf4
commit ca0d0d52f5
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/collectionWatch.js
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);
try {
// While MakerWorld is waiting for you to prove you're not a robot, the // 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. // 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'); const withFiles = Boolean(watch.fetch_files) && !downloadBlock.isBlocked('makerworld');
try {
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