Skip to content

Add candidate display aliases#681

Merged
SoundBlaster merged 2 commits into
mainfrom
codex/0212-candidate-display-aliases
Jul 10, 2026
Merged

Add candidate display aliases#681
SoundBlaster merged 2 commits into
mainfrom
codex/0212-candidate-display-aliases

Conversation

@SoundBlaster

@SoundBlaster SoundBlaster commented Jul 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Add proposal 0212 and deterministic, presentation-only candidate display aliases.
  • Link related task: Human-Friendly Candidate Aliases.

Motivation

  • Long generated candidate ids and constraint labels are difficult to scan in product review surfaces.
  • Consumers must not independently abbreviate canonical references.

Goals

  • Publish public-safe aliases without changing ids, refs, materialized paths, or promotion inputs.
  • Keep ontology aliases, routing aliases, gates, and Git authority out of scope.

Changes

  • Derive and validate aliases in candidate graph generation; preserve them through rerun materialization.
  • Expose alias index and topology endpoint aliases in Candidate Overview; use aliases in materialized review YAML.

Spec Evidence

Spec-ID: NO-SPEC-IMPACT
Proposal-ID: 0212
Spec-Impact: behavior
Spec-Rationale: This is a review-only producer and presentation contract over existing candidate graph identities; canonical spec ids and promotion semantics are unchanged.

Validation

  • Tests added/updated for changed behavior
  • Local checks passed

Commands run:

.venv/bin/python -m pytest tests/test_candidate_spec_graph.py tests/test_candidate_overview.py tests/test_candidate_spec_materialization.py tests/test_idea_to_spec_rerun_materialization.py -q
make test
make publish-bundle
make proposal-tracking-gate
make docc-sync

Results:

  • Focused tests: 52 passed.
  • Full suite: 1844 passed.
  • Public bundle safety gate: passed.

Risks / Notes

  • Backward compatibility impact: additive optional fields; older consumers continue to use title/id.
  • Migration/config changes required: none.
  • Known limitations: aliases are producer-derived and not operator-editable.

Checklist

  • PR title clearly describes the change
  • Scope is focused and minimal
  • Documentation updated (or N/A)
  • Workflow/artifact changes distinguish presence from readiness/status
  • Consumers prefer authoritative repaired/promoted/latest artifacts where relevant
  • Temporary or intermediate paths are scoped to the current run/task where relevant
  • No secrets or sensitive data added

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7f70bccfab

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/candidate_spec_graph.py
Comment thread tools/candidate_spec_graph.py
Comment thread tools/candidate_spec_materialization.py
Comment thread tools/candidate_spec_graph.py Outdated
Comment thread tools/candidate_spec_graph.py
@SoundBlaster SoundBlaster merged commit 51bde08 into main Jul 10, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant