Skip to content

Elevate GoalOS / Proof Gradient README, docs, figures, tables, badges, and validation#29

Open
MontrealAI wants to merge 1 commit into
mainfrom
codex/elevate-repository-to-institutional-public-foundation-muua8m
Open

Elevate GoalOS / Proof Gradient README, docs, figures, tables, badges, and validation#29
MontrealAI wants to merge 1 commit into
mainfrom
codex/elevate-repository-to-institutional-public-foundation-muua8m

Conversation

@MontrealAI

Copy link
Copy Markdown
Owner

Motivation

  • Raise the repository to an institutional public foundation posture by making the catalog the authoritative source-of-truth and preventing accidental public exposure of paid buyer artifacts.
  • Strengthen validation and CI so drift between README, docs, tables, figures, badges, and site releases is detected early and operators are guided to the autonomous GitHub Actions release path (v14 validation → Public Site Release v8).
  • Harden public-safety checks and documentation so the repo feels official, proof-bounded, enterprise-ready, and buyer-safe without changing production code or exposing paid deliverables.

Description

  • Updated the catalog source-of-truth by adding an explicit badge_inventory and the docs/tables/goalos_badge_inventory.csv entry in docs/data/goalos_catalog.yml so badges are verified by the catalog.
  • Hardened validation logic in scripts/validate_goalos_catalog.py to require the full institutional table set, all required figure .mmd/.svg pairs, all static badge SVGs, and exact governance workflow names, and to cross-check those entries against the catalog.
  • Strengthened scripts/validate_docs_tables_figures.py to include docs/tables/goalos_badge_inventory.csv and to verify the required static badges as part of docs/tables/figures validation.
  • Recorded the hardening pass and validation outcomes in docs/GOALOS_REPO_AUDIT.md so operators and reviewers can see the validation commands, test results, and non-regression assurances.

Testing

  • Ran the paid-file guard python scripts/check_no_paid_artifacts.py and it passed (allowed public AEP package only).
  • Ran site/docs validations python scripts/validate_goalos_public_site.py, python scripts/validate_docs_tables_figures.py, and python scripts/validate_goalos_catalog.py and they passed after bringing the catalog into alignment with validator expectations.
  • Executed the unit/integration suites with pytest (initially failed due to missing httpx2/httpx, installed via python -m pip install httpx2 httpx) and pytest then passed with 85 tests and 2 deprecation warnings.
  • Executed make test (unittest discovery) which passed (56 tests) and the Node-based GoalOS Cloud MVP check node site/app/goalos-cloud-mvp/tests/enterprise-core.test.mjs which also passed.

Codex Task

@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: 41b9dd776c

ℹ️ 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 on lines +176 to +178
for badge in REQUIRED_BADGES:
if f"badges/{badge}" not in cat:
errors.append(f"catalog badge_inventory missing badges/{badge}")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Require badge entries inside badge_inventory

In this new badge catalog check, the code searches the entire YAML text instead of the badge_inventory list, so a catalog can delete or rename badge_inventory: or leave these paths in comments/another section and still pass. Since this hardening is meant to make the catalog's badge inventory authoritative, validate the section itself (or parse YAML) before accepting these paths.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant