Switch to uv and prek, and enforce the shared lint suite #100

Activity

1. rtibbles

   added

   dependenciesPull requests that update a dependency filePull requests that update a dependency file

   github_actionsPull requests that update GitHub Actions codePull requests that update GitHub Actions code

   on Sep 26, 2026
2. rtibbles

   added the

   Task

   issue type on Sep 26, 2026
3. rtibblesbot commented on Oct 6, 2026

   

   rtibblesbot

   on Oct 6, 2026

   Contributor

   More actions

   The 13 ESLint errors on main are 12 no-console and one max-len, at 0ce46cf:

   • .github/scripts/generate-automation.js

     Lines 258 to 298 in 0ce46cf

     	...Object.entries(buildReusableWorkflows(automations, retiredSecrets)).map(([file, content]) => [
     	path.join(WORKFLOWS_DIR, file),
     	content,
     	]),
     	[TEMPLATE_PATH, buildTemplateYml(automations, secrets,
     	'Caller template: copy this file to .github/workflows/automation.yml in a consumer repo.\n' +
     	'# No edits are needed - the on: block is the exhaustive union of every enabled automation.')],
     	[DOGFOOD_PATH, buildTemplateYml(automations, secrets,
     	'This repo\'s own automation caller - .github acts as consumer zero and stays in sync automatically.')],
     	];
     	const targetPaths = new Set(targets.map(([target]) => target));
     	const stale = fs
     	.readdirSync(WORKFLOWS_DIR)
     	.filter((f) => f.startsWith(GROUP_PREFIX))
     	.map((f) => path.join(WORKFLOWS_DIR, f))
     	.filter((f) => !targetPaths.has(f));
     	if (check) {
     	let drifted = false;
     	for (const [target, content] of targets) {
     	const current = fs.existsSync(target) ? fs.readFileSync(target, 'utf8') : null;
     	if (current !== content) {
     	console.error(`Drift detected in ${path.relative(ROOT, target)} - run \`node scripts/generate-automation.js\``);
     	drifted = true;
     	}
     	}
     	for (const target of stale) {
     	console.error(`Stale ${path.relative(ROOT, target)} - run \`node scripts/generate-automation.js\``);
     	drifted = true;
     	}
     	process.exit(drifted ? 1 : 0);
     	}
     	for (const [target, content] of targets) {
     	fs.mkdirSync(path.dirname(target), { recursive: true });
     	fs.writeFileSync(target, content);
     	console.log(`Wrote ${path.relative(ROOT, target)}`);
     	}
     	for (const target of stale) {
     	fs.unlinkSync(target);
     	console.log(`Removed ${path.relative(ROOT, target)}`);

   • .github/scripts/sync-automation-template.js

     Lines 312 to 348 in 0ce46cf

     	console.log(`${r.repo.padEnd(26)} ${r.state.padEnd(20)} ${extra}`);
     	}
     	const problems = results.filter((r) => PROBLEM_STATES.includes(r.state));
     	const drifted = results.filter((r) => r.state !== 'in-sync');
     	console.log(`\n${results.length} consumers, ${drifted.length} not in sync, ${problems.length} needing attention.`);
     	for (const p of problems) {
     	console.log(`::error title=${p.repo}::${p.state}${p.detail ? `: ${p.detail}` : ''}`);
     	}
     	return problems.length;
     	}
     	// Both spellings, because an --only nobody parses is a full run against every
     	// consumer, which is the opposite of what it asks for.
     	function parseArgs(argv) {
     	const i = argv.findIndex((a) => a === '--only' || a.startsWith('--only='));
     	let only;
     	if (i !== -1) {
     	only = argv[i] === '--only' ? argv[i + 1] : argv[i].slice('--only='.length);
     	if (!only || only.startsWith('--')) throw new Error('--only needs a repo name');
     	}
     	return { dryRun: argv.includes('--dry-run'), only };
     	}
     	async function main() {
     	const token = process.env.GITHUB_TOKEN;
     	if (!token) {
     	console.error('GITHUB_TOKEN is not set.');
     	process.exit(1);
     	}
     	const template = fs.readFileSync(TEMPLATE_PATH, 'utf8');
     	const results = await run(httpApi(token), template, parseArgs(process.argv.slice(2)));
     	process.exit(report(results) ? 1 : 0);
     	}
     	if (require.main === module) {
     	main().catch((err) => {
     	console.error(err.message);

   • .github/scripts/sync-automation-template.test.js

     Lines 352 to 364 in 0ce46cf

     	const quiet = console.log;
     	console.log = () => {};
     	try {
     	const problems = report([
     	{ repo: 'a', state: 'in-sync' },
     	{ repo: 'b', state: 'opened', pr: 1 },
     	{ repo: 'c', state: 'declined', pr: 2 },
     	{ repo: 'd', state: 'error', detail: 'no access' },
     	{ repo: 'e', state: 'toolchain-conflict', pr: 3 },
     	]);
     	assert.equal(problems, 2, 'error and toolchain-conflict each need attention');
     	} finally {
     	console.log = quiet;

   Until they are fixed, yarn lint exits 1, so #109's "yarn lint passes" criterion can't be met.

   ---

   Written by rtibblesbot, an LLM-based coding agent.