feat: add interactive profile design wizard

[leo-guinan]

Implements #13 with class selection, companion bundles, and generated profile.params.yaml. Includes tests/ and docs.

• Class config in templates/wizard/classes.yaml
• Bundles in templates/wizard/bundles.yaml
• Script in scripts/profile_wizard.py
• README path added

Changes based on maintainer review

• Added --force flag to overwrite existing output safely
• load_yaml now fails loudly on missing config files
• Added model compatibility test proving wizard params map to compatible config.yaml
• Added end-to-end smoke test: wizard → generate_profile.py → validated profile
• Copied wizard fixtures into tests/fixtures/wizard/ to remove symlink dependency

Verification

• python3 -m unittest tests.test_profile_wizard -v — 6/6 passed
• make validate — passed
• make generate-smoke — passed
• make release-check — passed for version 0.7.0

Bounty payout address

Solana: FFAdvcr2CUPbaQSypK3c8WfQo3SuyBh5YKrtzZRSvg34

Closes #13.

[codegraphtheory]

Thanks @leo-guinan for the contribution. I am starting a maintainer review for this PR now.

Selection note: this is the PR I am reviewing for you as a first-time contributor candidate on codegraphtheory/hermes-profile-template. I am checking it against the linked issue #13, repo quality gates, and whether it overlaps with other active PRs.

Verification so far: I ran python3 -m unittest tests/test_profile_wizard.py. It passed: 4 tests OK.

Bounty metadata: This closes a bounty issue, so please add a Solana wallet address to the PR body before payout review.

[codegraphtheory]

Review note for scripts/profile_wizard.py:17

DEFAULT_OUTPUT = Path("profile.params.yaml") writes into whatever directory the caller happens to be in. That is risky for a profile authoring tool because an accidental run from a repo root can overwrite an existing params file. Please default to an explicit confirmation or require --force when the output already exists.

Suggested change:

def write_params(path: Path, params: dict, *, force: bool = False) -> None:
    if path.exists() and not force:
        raise SystemExit(f"Refusing to overwrite {path}. Re-run with --force or choose --output.")
    path.write_text(yaml.safe_dump(params, sort_keys=False), encoding="utf-8")

[codegraphtheory]

Review note for scripts/profile_wizard.py:20

load_yaml() returns {} for a missing config file. That makes a broken install look like an empty wizard instead of a clear setup failure. Please fail loudly for the default classes and bundles files, and only allow empty data when a user intentionally points at an optional file.

Suggested change:

def load_yaml(path: Path) -> dict:
    if not path.exists():
        raise FileNotFoundError(f"Wizard config not found: {path}")
    data = yaml.safe_load(path.read_text(encoding="utf-8"))
    if not isinstance(data, dict):
        raise ValueError(f"Wizard config must be a YAML mapping: {path}")
    return data

[codegraphtheory]

Review note for scripts/profile_wizard.py:61

The generated params use model_default, but current Hermes profile config compatibility expects model.provider plus model.model downstream. Please either emit fields that the existing generator already maps safely, or add a test that proves the wizard output compiles into a compatible config.yaml.

Suggested assertion:

params = build_params(engineer, None)
self.assertEqual(params["model_provider"], "openrouter")
self.assertIn("/", params["model_default"])

[codegraphtheory]

Review note for tests/fixtures/wizard:1

This fixture is a symlink-style text file rather than a real fixture directory in the diff. That can be brittle on platforms and makes the tests depend on production templates. Please copy minimal fixture YAML into tests/fixtures/wizard/ or have the tests read from templates/wizard/ directly with an explicit comment.

Suggested layout:

tests/fixtures/wizard/classes.yaml
tests/fixtures/wizard/bundles.yaml

[codegraphtheory]

Review note for scripts/profile_wizard.py:152

Please add a non-interactive smoke test that writes to a temporary output path and then runs scripts/generate_profile.py --params <that file> --output <tmpdir>. The current tests cover build_params, but not the end-to-end contract promised in the README.

[codegraphtheory]

Thanks again for the work here. I will be happy to accept this PR once the requested updates above are addressed, the branch is kept scoped to #13, and the verification commands in the PR body are refreshed after the changes.

If this is a bounty PR, please make sure the Solana wallet address is present and not a placeholder before payout review.

[codegraphtheory]

Thanks @leo-guinan. I integrated the relevant parts of this contribution into #53 and merged it to main as commit f9e7d88.

You were included as a Co-authored-by: contributor in the merged squash commit so GitHub can credit you in repository history.

Wallet metadata was captured from your PR body for payout review.

I am closing this PR as superseded by the merged maintainer integration to keep at most one accepted path per first-time contributor and avoid duplicate implementations for the same issues.

[codegraphtheory]

Superseded by merged maintainer integration #53 with co-author credit.

[codegraphtheory]

Paid. Thank you for the contribution.

Wallet: FFAdvcr2CUPbaQSypK3c8WfQo3SuyBh5YKrtzZRSvg34
Transaction: https://solscan.io/tx/3qGbbxGr8pQLoup6wRDNHqDWkhaZFjqDMmQRSP2ZQ1XaVBYjkhCfcrmeSSFJ8uA5a4bfwVyRUsB91RGHyTe9ymE4