🚀 Feature: Make "locator" Options required: owner and repository

[JoshuaKGoldberg]

Bug Report Checklist

• I have pulled the latest main branch of the repository.
• I have searched for related issues and found none that matched my issue.

Overview

Creating a new repository on GitHub requires a "locator" consisting of:

1. Owner: e.g. "JoshuaKGoldberg"
2. Repository: e.g. "create"

Right now there is have some hardcoded logic to create a new repository in runPreset:

create/packages/create/src/runners/runPreset.ts

Lines 59 to 66 in 9abcd2e

	if (settings.mode === "initialize") {
	// TODO: Hardcode owner and repository existing in options?
	await createRepositoryOnGitHub(
	options as unknown as CreationOptions,
	system.runner,
	preset.base.template,
	);
	}

create/packages/create/src/runners/runPreset.ts

Lines 70 to 75 in 9abcd2e

	if (settings.mode === "initialize") {
	await createTrackingBranches(
	options as unknown as CreationOptions,
	system.runner,
	);
	}

That logic assumes owner and repository must exist on the Preset's Base's Options. Which, given that all Presets target GitHub repositories right now, is actually a fair assumption. And baking these in as automatically included, non-overridable Options would be a nice bit of work reduction for implementers.

Additional Info

In some far-off distant future, this engine will likely support repository creation targets other than GitHub. This will need to be revisited then. But for now, since create-typescript-app is the main feature goal and CTA only supports GitHub, this is fine.

💖

[JoshuaKGoldberg]

I explored this in #72. It's really hacky to make them requested altogether, because otherwise Options are allowed to be any Record<string, ...> including default/optional types.

The compromise I went with is requiring the options exist when used to initialize:

create/packages/create/src/cli/initialize/assertOptionsForInitialize.ts

Lines 6 to 11 in 78bc44b

	for (const key of ["owner", "repository"]) {
	if (typeof (options as Record<string, unknown>)[key] !== "string") {
	throw new Error(
	`To run with --mode initialize, the Template must have a --${key} Option of type string.`,
	);
	}

I haven't yet fleshed out migration mode. When that happens I might revisit this. But until then, declining.