Skip to content

docs: state the open-execution wedge on the acquisition surfaces - #88

Merged
kmadan merged 1 commit into
mainfrom
docs/adoption-readme
Aug 30, 2026
Merged

docs: state the open-execution wedge on the acquisition surfaces#88
kmadan merged 1 commit into
mainfrom
docs/adoption-readme

Conversation

@kmadan

@kmadan kmadan commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

🚀 Overview

Rewrites the three surfaces a prospective user meets first — README.md, the
PyPI description, and aicertify --help — so they state what distinguishes
AICertify (open, versioned Rego that a third party can re-run against the same
inputs) rather than listing output formats. Includes three accuracy corrections.

No behaviour changes. The only aicertify/cli.py edits are the module docstring
and the parser description string.

📜 Changes Made

  • README.md — leads with open execution and evidence; adds jump navigation;
    makes reproducibility the primary differentiation instead of framework breadth;
    states the declared-facts vs measured-evidence distinction explicitly; changes
    the canonical example to aicertify evaluate; describes the demo's fail-closed
    result as intended behaviour rather than leaving it to read as a defect.
  • README-pypi.md — resynchronised with the README; stale claims removed.
  • pyproject.toml — description states the same wedge, so PyPI search results
    carry it.
  • aicertify/cli.py — docstring and help text corrected (see below).

main carried two ## Why AICertify sections, at L189 and L343. The second is
replaced by ## What makes an AICertify result reproducible. Heading count is
unchanged at 26 and no heading is duplicated.

🔧 Accuracy corrections

Claim Was Now Basis
Policy count (README.md) 91 92 docs/coverage/coverage.json totals in the pinned GOPAL tree
Policy count (README-pypi.md) 85 92 same
CLI subcommands "Two subcommands" five, named add_parser count in cli.py
Demo duration "10-second demo" removed unmeasured claim

The GOPAL tree pinned at aicertify/opa_policies reports
policies: 92, policies_with_tests: 92, policies_with_empty_input_test: 92.
The five registered subcommands are demo, evaluate, explain,
init-contract and score-card.

🎯 How to Test

python -m aicertify.cli --help          # renders all five subcommands
python -c "import tomllib; tomllib.load(open('pyproject.toml','rb'))"

Verified locally: cli.py compiles, --help renders the five subcommands,
pyproject.toml parses, the added relative link
(docs/integrations/github-actions.md) resolves, all five jump-navigation
anchors resolve to real headings, and pre-commit (ruff, black, whitespace)
passes.

📚 Documentation

  • ✅ Updated README.md, README-pypi.md, pyproject.toml description, CLI help
  • ✅ No API or policy changes

📄 Notes for reviewers

The submodule pointer aicertify/opa_policies is deliberately not touched by
this PR. The 92 figure holds at the pin already on main, so the correction does
not depend on a bump.

Branch docs/technical-voice (5ab4d11) is pushed with no open PR and overlaps
this one on README.md, README-pypi.md and cli.py. This PR is based on
main, not on that branch. That branch needs a separate decision — see the
discussion on the PR.

@github-actions github-actions Bot added 📝 documentation Improvements or additions to documentation 🚀 enhancement New feature or request labels Aug 30, 2026
@kmadan
kmadan requested a review from Prinevo August 30, 2026 04:07
The README, the PyPI description and the CLI help are the first three surfaces
a prospective user meets, and each described AICertify by its output formats
rather than by what distinguishes it: the policy logic is open, versioned Rego
that a third party can re-run against the same inputs.

README.md now leads with open execution and evidence, carries a jump navigation
for the four questions a first-time reader asks, and makes reproducibility the
primary differentiation rather than framework breadth. The declared-facts and
measured-evidence distinction is stated explicitly, since a supplied number is
an assertion unless the evaluator evidence behind it is supplied too. The
canonical example becomes `aicertify evaluate`, and the demo's fail-closed
result is described as intended behaviour rather than left to look like a
defect. main carried two "Why AICertify" sections; the second is replaced by
"What makes an AICertify result reproducible".

README-pypi.md is resynchronised with the README, and pyproject's description
states the same wedge so PyPI search results carry it.

Three accuracy corrections:

- The policy count read 91 in README.md and 85 in README-pypi.md. The GOPAL
  tree pinned at aicertify/opa_policies reports 92 policies, all 92 with tests
  and empty-input tests, per docs/coverage/coverage.json totals.
- cli.py documented "Two subcommands". The parser registers five: demo,
  evaluate, explain, init-contract and score-card.
- The "10-second demo" claim in cli.py help and README-pypi.md is unmeasured
  and is removed.

No behaviour changes; the only cli.py edits are the module docstring and the
parser description string. Verified: cli.py compiles and `--help` renders the
five subcommands, pyproject.toml parses, every added relative link and all five
jump-navigation anchors resolve, and no heading is duplicated.
@kmadan
kmadan force-pushed the docs/adoption-readme branch from 62a58da to 38eef59 Compare August 30, 2026 04:20
@kmadan
kmadan merged commit c8c3c93 into main Aug 30, 2026
6 checks passed
kmadan added a commit that referenced this pull request Aug 30, 2026
…-additive

docs: carry the technical-voice pass into the surfaces #88 did not touch
@kmadan
kmadan deleted the docs/adoption-readme branch August 30, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📝 documentation Improvements or additions to documentation 🚀 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants