Skip to content

ci(publication): enforce placeholder policy remotely#2

Merged
stacknil merged 1 commit into
mainfrom
stacknil/placeholder-policy-ci
Jul 14, 2026
Merged

ci(publication): enforce placeholder policy remotely#2
stacknil merged 1 commit into
mainfrom
stacknil/placeholder-policy-ci

Conversation

@stacknil

Copy link
Copy Markdown
Owner

Summary

  • run the canonical placeholder validator in the main Markdown Validation workflow
  • trigger the workflow when the validator implementation changes
  • keep the existing manual artifact audit as a separate non-blocking maintenance surface

Design decision

Use the existing full-corpus validator as the remote gate. The current 157-note corpus is clean, so no debt exception or duplicate changed-file implementation is required.

Main risk

This check enforces placeholder vocabulary only. It does not replace human publication review, secret scanning, or authorization review.

Compatibility impact

No note schema or public content changes. Contributors may now receive a failing PR check for noncanonical placeholders that local pre-commit already rejected.

Rollback path

Revert this single workflow commit to restore the previous CI behavior.

Verification

  • python scripts/render_tags_doc.py --check
  • python scripts/render_readme_snapshot.py --check
  • python scripts/check_pattern_library.py
  • python scripts/check_placeholders.py
  • python scripts/check_markdown.py
  • python -m pre_commit run --files .github/workflows/markdown-lint.yml
  • git diff --check

@stacknil

Copy link
Copy Markdown
Owner Author

Post-CI diff review complete.

  • Design decision: run the existing full-corpus placeholder validator in the main Markdown Validation job and trigger that workflow when the validator changes.
  • Main risk: this gate validates placeholder vocabulary only; human publication review, authorization checks, and secret scanning remain separate responsibilities.
  • Compatibility impact: no content or schema changes. Remote CI now enforces a policy that local pre-commit already enforced.
  • Rollback path: revert commit 309c6ce to remove the workflow step and trigger entries.

Evidence: the reviewed local and remote heads match; the diff is one workflow file with five added lines; git diff --check and privacy scanning pass; both workflow runs are green; and the GitHub job log confirms the new placeholder-validation step completed successfully.

@stacknil stacknil merged commit da79f73 into main Jul 14, 2026
4 checks passed
@stacknil stacknil deleted the stacknil/placeholder-policy-ci branch July 14, 2026 06:15
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