Merge thumbnail validation fix
All checks were successful
Validate Plugin PR / validate (pull_request) Successful in 9s
All checks were successful
Validate Plugin PR / validate (pull_request) Successful in 9s
This commit is contained in:
@@ -104,24 +104,7 @@ const changedFiles = process.argv[3]?.split('\n').filter(f => f.trim()) || [];
|
|||||||
if (!prAuthor) {
|
if (!prAuthor) {
|
||||||
console.error('❌ Error: PR author not provided');
|
console.error('❌ Error: PR author not provided');
|
||||||
process.exit(1);
|
process.exit(1);
|
||||||
}// Check thumbnail if provided (optional)
|
}
|
||||||
if (plugin.thumbnail) {
|
|
||||||
try {
|
|
||||||
new URL(plugin.thumbnail);
|
|
||||||
|
|
||||||
// Validate thumbnail meets requirements
|
|
||||||
try {
|
|
||||||
const thumbInfo = await validateThumbnail(plugin.thumbnail);
|
|
||||||
console.log(` ✓ Thumbnail: ${thumbInfo.format} ${thumbInfo.width}x${thumbInfo.height} (${(thumbInfo.size / 1024).toFixed(1)}KB)`);
|
|
||||||
} catch (thumbErr) {
|
|
||||||
errors.push(`❌ ${file}: Thumbnail validation failed: ${thumbErr.message}`);
|
|
||||||
}
|
|
||||||
} catch {
|
|
||||||
errors.push(`❌ ${file}: Invalid thumbnail URL: ${plugin.thumbnail}`);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
console.log(`\n🔍 Validating PR from: ${prAuthor}`);
|
console.log(`\n🔍 Validating PR from: ${prAuthor}`);
|
||||||
console.log(`📝 Changed files: ${changedFiles.length}`);
|
console.log(`📝 Changed files: ${changedFiles.length}`);
|
||||||
@@ -235,6 +218,23 @@ for (const file of changedFiles) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Check thumbnail if provided (optional)
|
||||||
|
if (plugin.thumbnail) {
|
||||||
|
try {
|
||||||
|
new URL(plugin.thumbnail);
|
||||||
|
|
||||||
|
// Validate thumbnail meets requirements
|
||||||
|
try {
|
||||||
|
const thumbInfo = await validateThumbnail(plugin.thumbnail);
|
||||||
|
console.log(` ✓ Thumbnail: ${thumbInfo.format} ${thumbInfo.width}x${thumbInfo.height} (${(thumbInfo.size / 1024).toFixed(1)}KB)`);
|
||||||
|
} catch (thumbErr) {
|
||||||
|
errors.push(`❌ ${file}: Thumbnail validation failed: ${thumbErr.message}`);
|
||||||
|
}
|
||||||
|
} catch {
|
||||||
|
errors.push(`❌ ${file}: Invalid thumbnail URL: ${plugin.thumbnail}`);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
if (errors.length === 0) {
|
if (errors.length === 0) {
|
||||||
added.push(pluginId);
|
added.push(pluginId);
|
||||||
console.log(`✅ Valid plugin: ${plugin.name}`);
|
console.log(`✅ Valid plugin: ${plugin.name}`);
|
||||||
|
|||||||
Reference in New Issue
Block a user