change unity-course.php prompts

[simonLeary42]

before:

• prompt for "course ID" (example: cs123)
• prompt for "cn" (example: cs123_org1_test)
• extract org from cn

after:

• prompt for "course ID" (example: cs123)
• prompt for org (example: org1_test)

[Copilot]

Pull request overview

This PR updates the unity-course.php worker’s interactive prompts to ask for the organization directly (instead of asking for a full cn and extracting the org), and adjusts the functional test to match the new prompt sequence.

Changes:

• Removed cn-based org extraction and now prompts for org explicitly.
• Added basic input validation for course ID and organization before constructing the cn.
• Updated WorkerUnityCourseTest stdin inputs to reflect the new prompt order.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
workers/unity-course.php	Changes the prompt flow to ask for org directly, constructs cn from course ID + org, and adds input validation.
test/functional/WorkerUnityCourseTest.php	Updates functional test stdin lines to match the new worker prompt sequence.

Suppressed comments (1)

workers/unity-course.php:34

• The organization validation reports “contains invalid characters” when the input is empty. Adding an explicit required/empty check first would produce a more accurate error for missing input.

$org_gid = strtolower(trim(readline("Please enter the organization (example: umass_edu): ")));
if (!_preg_match("/^[a-z0-9_]+$/", $org_gid)) {
    _die("error: organization '$org_gid' contains invalid characters", 1);
}

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.