Skip to content

Update bundled Repository Governance snapshot#48

Merged
bigsmartben merged 2 commits into
mainfrom
community/repository-governance-ssot-index
Jul 3, 2026
Merged

Update bundled Repository Governance snapshot#48
bigsmartben merged 2 commits into
mainfrom
community/repository-governance-ssot-index

Conversation

@bigsmartben

Copy link
Copy Markdown
Owner

Description

Update the bundled repository-governance extension snapshot from bigsmartben/spec-kit-agent-governance source commit 42f0cb04891a29a4c05973b7fa5e746a7e0f8dd4.

This pulls in the SSOT index projection changes, explicit Zed target mapping, bounded repository evidence scanning, narrower generated target content, and matching bundled README/changelog updates. The bundled install URL now points at the exact source commit archive used for this snapshot instead of an older tag.

Testing

  • Tested locally with uv run specify --help
  • Ran existing tests with uv sync && uv run pytest
  • Tested with a sample project (if applicable)

Validation performed:

  • uv run python scripts/community/validate_integration.py --repo-root . --branch community/repository-governance-ssot-index — passed
  • .\.venv\Scripts\python -m py_compile extensions\repository-governance\scripts\generate_repository_governance.py — passed
  • .\.venv\Scripts\python -m pytest tests\integrations\test_cli.py::TestGitExtensionOptIn::test_community_extensions_and_workflow_preset_auto_installed tests\integrations\test_cli.py::TestGitExtensionOptIn::test_no_git_keeps_community_defaults tests\test_arch_templates.py tests\integrations\community_defaults.py — 8 passed
  • .\.venv\Scripts\python -m pytest — attempted, timed out after 10 minutes and ended with OSError: [Errno 22] Invalid argument while flushing stdout, so no full-suite pass is claimed

Community Catalog Submission

AI Disclosure

  • I did not use AI assistance for this contribution
  • I did use AI assistance (describe below)

Prepared by OpenAI Codex (GPT-5) on behalf of @bigsmartben. Codex synchronized the bundled extension snapshot, updated metadata, ran validation, committed, pushed, and opened this PR under direct user instruction.

Assisted-by: OpenAI Codex (GPT-5, autonomous)

@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: 958bc06388

ℹ️ 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 +485 to +487
if child.is_dir():
dirs.append(child)
next_dirs.append(child)

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 Skip symlinked directories during project walks

With a repository-local symlinked directory, for example loop -> .. or a linked package outside the repo, Path.is_dir() follows the symlink and project_walk() pushes it onto the traversal stack. Because every bounded evidence helper calls project_files()/project_dirs() before applying depth limits, the generator can recurse outside the project or hang on symlink cycles instead of producing the governance projection; skip symlinked directories or verify resolved children remain under root before appending them.

Useful? React with 👍 / 👎.

"docs/*agent*governance*.md",
"docs/**/*agent*governance*.md",
]
CODE_STYLE_SSOT_FILES = [

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 Include style config files in the Code Style SSOT index

When the only code-style SSOT is an explicit tool config such as .prettierrc, eslint.config.js, ruff.toml, mypy.ini, or style settings in pyproject.toml, this new whitelist makes code_style_ssot_refs() report Code Style as missing. The generated projection then tells agents to ask for NEEDS_CLARIFICATION:CODE_STYLE instead of reading the actual formatter/linter config, so include the existing CODE_STYLE_FILES entries, or equivalent parsed pyproject.toml refs, in the index source refs.

Useful? React with 👍 / 👎.

relative = path.relative_to(root)
if len(relative.parts) - 1 > max_parent_depth:
continue
if any(relative.match(pattern) for pattern in patterns):

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 Match nested SSOT glob refs recursively

For SSOT docs nested more than one directory under docs/ or adr/, for example docs/platform/backend/architecture-overview.md, Path.match('docs/**/architecture*.md') does not match recursively the way these patterns imply, so bounded_glob_files() misses valid refs even within EVIDENCE_SCAN_MAX_PARENT_DEPTH. Use root.glob or fnmatch with explicit depth handling so the ** entries actually cover the documented nested SSOT locations.

Useful? React with 👍 / 👎.

Allow fork-hosted community integration PRs to reuse the pr-template route without requiring upstream maintainer approval, while preserving the approval requirement for github/spec-kit.

Assisted-by: OpenAI Codex (GPT-5, autonomous)

Copy link
Copy Markdown
Owner Author

Posted on behalf of @bigsmartben by OpenAI Codex (GPT-5).

Updated the community integration validation workflow in commit 8ef2ab9. The workflow now passes the current repository name into validate_integration.py, so fork-hosted integration PRs can use the pr-template route without requiring an upstream maintainer-approval URL, while github/spec-kit still enforces that approval requirement.

Validation run locally:

  • .\.venv\Scripts\python -m pytest tests\test_community_validate_integration.py -v — 4 passed
  • uv run python -m pytest tests\test_community_validate_integration.py -v — 4 passed
  • PR body validation for bigsmartben/spec-kit — passed
  • PR body validation for github/spec-kit without approval URL — failed as expected
  • .\.venv\Scripts\python -m py_compile scripts\community\validate_integration.py tests\test_community_validate_integration.py — passed

GitHub Actions status after push: validate-community-integration, ruff, lint, smoke, and CodeQL passed; pytest matrix was still running at last check.

@bigsmartben bigsmartben merged commit 9031f3c into main Jul 3, 2026
14 checks passed
@bigsmartben bigsmartben deleted the community/repository-governance-ssot-index branch July 3, 2026 10:58
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