Skip to content

ci: validate documentation and run the Scribe suite on PRs - #4

Open
logbie wants to merge 2 commits into
devfrom
sawako/log-16-ci-readiness
Open

logbie wants to merge 2 commits into
devfrom
sawako/log-16-ci-readiness

Conversation

@logbie

@logbie logbie commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Scribe has no Actions workflow on baseline b3c6673. Add observable documentation and runtime checks for main/dev PRs under the approved LOG-16 readiness assignment.

Changes

Documentation checks validates changed Markdown whitespace, nonempty README/SYNTAX/DESIGN entry points and syntax of documented shell runners. It does not validate Markdown links, prose accuracy or accessibility; those remain review responsibilities. Scribe tests runs the existing complete WFL test file in a disposable copy with no network or credentials, recording the resolved nightly image digest and runtime version.

Compatibility and risk

R1: CI only. No application, release/deploy, credential, branch-setting or production changes. Existing Scriptorium provisioning uses the same official nightly image; each run records its resolved digest. PRs require both successful jobs before adding these names to a required-check proposal. Existing controls remain unchanged.

Validation

Baseline has no workflow: no failing Actions run is possible before adding one. Existing Scribe behavioral regressions are retained, without skips or relaxed assertions. YAML parsing, separate bash -n calls for each runner, and diff whitespace validation pass locally. No local WFL/Docker runtime is available; both jobs pass on current head 87d26e879dc590e25477ccc5825714b623de097a: Actions. This adds check wiring, not new validation algorithms.

Checklist

  • Own feature branch targets dev, as authorized for the governance rollout.
  • Read-only token; checkout credential persistence disabled; no secrets consumed.
  • Both current-head Actions jobs pass.
  • Yomi review and bot findings dispositioned.
  • Independent approval satisfied. No merge or bypass authorized.

Devin finding corrected: the initial multi-file bash command checked only the first file. A local negative fixture demonstrated old exit 0 versus independent syntax-check exit 2; fixed both commands and reran Actions. No failing Actions run is claimed for that local reproduction.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c952f4de-bb79-47fa-b299-aab81fcd9fac

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration 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.

Devin Review found 1 potential issue.

Devin Review

Comment thread .github/workflows/ci.yml Outdated
Co-Authored-By: Paperclip <noreply@paperclip.ing>
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.

2 participants