style(ncl): format Nickel files with nickel format by hyperpolymath · Pull Request #195 · hyperpolymath/rsr-template-repo

hyperpolymath · 2026-09-22T13:26:43Z

Formatting only — all 11 genuine Nickel files in the template fail
nickel format --check.

This is where the defect originates. Every repo seeded from this template
inherits these files, so each one goes red on the ci-pipeline Nickel gate over
boilerplate it did not write. cicd-squabbler is the measured case
(hyperpolymath/cicd-squabbler#106) — it went 100% red on template files, never on
its own source. Fixing the template stops the defect propagating to new seeds.

The other half of that red was a defect in the shared gate itself, fixed in
hyperpolymath/standards#985: it globbed *.ncl, so it also checked the 18
*.k9.ncl k9 contracts here — a different format (K9! sentinel on line 1)
that can never pass nickel typecheck. Those 18 files are untouched by this PR.

Verification

Run with the pipeline's own pinned nickel 1.18.0 (sha256-verified from the
workflow), not a local build — formatter output differs between releases.

control	result
`nickel pprint-ast` byte-identical before vs after	11 / 11
SPDX + copyright headers unchanged	11 / 11
total comment lines	356 before, 356 after
files touched outside genuine `*.ncl`	0
`nickel format --check` after	passes on all 11

The AST comparison is the point: this is provably a cosmetic reformat, not a
semantic change.

Not an auto-fix

No --fix runs in CI. The pipeline still uses --check only; this is a
deliberate commit in a reviewable PR.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ji1bq3TypfycfUPAR7hSxR

All 11 genuine Nickel files in the template fail `nickel format --check`, so every repo seeded from this template inherits the defect and goes red on the ci-pipeline Nickel gate over boilerplate it did not write. Fixing it here stops the defect propagating to new seeds. Formatting only. Verified semantics-preserving by comparing the parsed AST: `nickel pprint-ast` output is byte-identical for 11 of 11 files before and after. SPDX and copyright headers are unchanged, and the total comment-line count is identical (356 before, 356 after). Run with the pipeline's own pinned nickel 1.18.0 (sha256-verified), because formatter output differs between releases. The 18 `*.k9.ncl` files are untouched: they are k9 contracts, not Nickel source, and are excluded by hyperpolymath/standards#985. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ji1bq3TypfycfUPAR7hSxR

coderabbitai · 2026-09-22T13:26:59Z

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: b4fd5f53-6fda-4c92-9459-22c1689a6df3

📥 Commits

Reviewing files that changed from the base of the PR and between 910d0b3 and 2eba87d.

📒 Files selected for processing (11)

.machine_readable/arrival-pack/arrival-pack.ncl
.machine_readable/coaptation/coapt.ncl
.machine_readable/coaptation/grades.ncl
.machine_readable/coaptation/witness-map.ncl
.machine_readable/contractiles/_base.ncl
.machine_readable/contractiles/adjust/adjust.ncl
.machine_readable/contractiles/bust/bust.ncl
.machine_readable/contractiles/dust/dust.ncl
.machine_readable/contractiles/intend/intend.ncl
.machine_readable/contractiles/must/must.ncl
.machine_readable/contractiles/trust/trust.ncl

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout. (38)

GitHub Check: Analyze (actions)
GitHub Check: Analyze (javascript-typescript)
GitHub Check: Analyze (rust)
GitHub Check: governance / Trusted-base reduction policy
GitHub Check: governance / Live Actions policy (credentialed advisory)
GitHub Check: governance / Actions lockfile verify
GitHub Check: governance / Allowlist Preflight
GitHub Check: governance / Guix packaging policy (Nix retired)
GitHub Check: governance / Language / package anti-pattern policy
GitHub Check: governance / Check Workflow Staleness
GitHub Check: governance / Security policy checks
GitHub Check: governance / Exemption ratchet
GitHub Check: governance / Debt ratchet
GitHub Check: governance / Workflow security linter
GitHub Check: governance / Licence consistency
GitHub Check: governance / Well-Known (RFC 9116 + RSR)
GitHub Check: governance / Code quality + docs
GitHub Check: call-estate-audit / estate-audit
GitHub Check: 🟡 CHECK: Validate K9 contracts
GitHub Check: scan / shell-secrets
GitHub Check: scan / gitleaks
GitHub Check: scan / rust-secrets
GitHub Check: hypatia / Hypatia Neurosymbolic Analysis
GitHub Check: estate-rules
GitHub Check: rust-ci / Detect Cargo.toml
GitHub Check: Validate DEED manifests
GitHub Check: Hypatia neurosymbolic scan
GitHub Check: Dogfooding compliance summary
GitHub Check: 🔴 GATE: Empty-linter (invisible characters)
GitHub Check: lint
GitHub Check: check
GitHub Check: Runtime Policy
GitHub Check: Validate DEED manifests
GitHub Check: Validate eclexiaiser manifest
GitHub Check: RSR oracle — dogfood this repo
GitHub Check: Canon lockstep
GitHub Check: Validate K9 contracts
GitHub Check: 🟡 CHECK: Groove manifest check

⚠️ CI failures not shown inline (14)

GitHub Actions: Governance / 4_governance _ Actions lockfile verify.txt: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run set -uo pipefail
 �[36;1mset -uo pipefail�[0m
 �[36;1mif [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then�[0m
 �[36;1m  SRC=scripts�[0m
 �[36;1m  echo "Using this repository's own gate + verifier (standards self-lint)."�[0m
 �[36;1melse�[0m
 �[36;1m  SRC=.standards-lock/scripts�[0m
 �[36;1mfi�[0m
 �[36;1mfor f in check-actions-lock-gate.sh update-actions-lock.sh; do�[0m
 �[36;1m  if [ ! -f "$SRC/$f" ]; then�[0m
 �[36;1m    echo "::error::actions-lock gate: $f not found in $SRC (standards checkout at the explicit helper pin failed?)"�[0m

GitHub Actions: Governance / governance _ Actions lockfile verify: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run set -uo pipefail
 �[36;1mset -uo pipefail�[0m
 �[36;1mif [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then�[0m
 �[36;1m  SRC=scripts�[0m
 �[36;1m  echo "Using this repository's own gate + verifier (standards self-lint)."�[0m
 �[36;1melse�[0m
 �[36;1m  SRC=.standards-lock/scripts�[0m
 �[36;1mfi�[0m
 �[36;1mfor f in check-actions-lock-gate.sh update-actions-lock.sh; do�[0m
 �[36;1m  if [ ! -f "$SRC/$f" ]; then�[0m
 �[36;1m    echo "::error::actions-lock gate: $f not found in $SRC (standards checkout at the explicit helper pin failed?)"�[0m

GitHub Actions: Governance / 5_governance _ Security policy checks.txt: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run FAILED=false
 �[36;1mFAILED=false�[0m
 �[36;1mWEAK_CRYPTO=$(grep -rE 'md5\(|sha1\(' --include="*.py" --include="*.rb" --include="*.js" --include="*.ts" --include="*.go" --include="*.rs" . 2>/dev/null | grep -v 'checksum\|cache\|test\|spec' | head -5 || true)�[0m
 �[36;1mif [ -n "$WEAK_CRYPTO" ]; then�[0m
 �[36;1m  echo "::warning::Weak crypto (MD5/SHA1) detected — ADVISORY, does not fail this job. Use SHA256+:"�[0m
 �[36;1m  echo "$WEAK_CRYPTO"�[0m
 �[36;1mfi�[0m
 �[36;1mHTTP_URLS=$(grep -rE 'http://[^l][^o][^c]' --include="*.py" --include="*.js" --include="*.ts" --include="*.go" --include="*.rs" --include="*.yaml" --include="*.yml" . 2>/dev/null | grep -v 'localhost\|127.0.0.1\|example\|test\|spec' | head -5 || true)�[0m
 �[36;1mif [ -n "$HTTP_URLS" ]; then�[0m
 �[36;1m  echo "::warning::HTTP URLs found — ADVISORY, does not fail this job. Use HTTPS:"�[0m
 �[36;1m  echo "$HTTP_URLS"�[0m
 �[36;1mfi�[0m
 �[36;1mSECRETS=$(grep -rEi '(api_key|apikey|secret_key|password)\s*[=:]\s*["\x27][A-Za-z0-9+/=]{20,}' --include="*.py" --include="*.js" --include="*.ts" --include="*.go" --include="*.rs" --include="*.env" . 2>/dev/null | grep -v 'example\|sample\|test\|mock\|placeholder' | head -3 || true)�[0m
 �[36;1mif [ -n "$SECRETS" ]; then�[0m
 �[36;1m  echo "::error::Potential hardcoded secrets detected — this FAILS the job:"�[0m

GitHub Actions: Governance / governance _ Security policy checks: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run FAILED=false
 �[36;1mFAILED=false�[0m
 �[36;1mWEAK_CRYPTO=$(grep -rE 'md5\(|sha1\(' --include="*.py" --include="*.rb" --include="*.js" --include="*.ts" --include="*.go" --include="*.rs" . 2>/dev/null | grep -v 'checksum\|cache\|test\|spec' | head -5 || true)�[0m
 �[36;1mif [ -n "$WEAK_CRYPTO" ]; then�[0m
 �[36;1m  echo "::warning::Weak crypto (MD5/SHA1) detected — ADVISORY, does not fail this job. Use SHA256+:"�[0m
 �[36;1m  echo "$WEAK_CRYPTO"�[0m
 �[36;1mfi�[0m
 �[36;1mHTTP_URLS=$(grep -rE 'http://[^l][^o][^c]' --include="*.py" --include="*.js" --include="*.ts" --include="*.go" --include="*.rs" --include="*.yaml" --include="*.yml" . 2>/dev/null | grep -v 'localhost\|127.0.0.1\|example\|test\|spec' | head -5 || true)�[0m
 �[36;1mif [ -n "$HTTP_URLS" ]; then�[0m
 �[36;1m  echo "::warning::HTTP URLs found — ADVISORY, does not fail this job. Use HTTPS:"�[0m
 �[36;1m  echo "$HTTP_URLS"�[0m
 �[36;1mfi�[0m
 �[36;1mSECRETS=$(grep -rEi '(api_key|apikey|secret_key|password)\s*[=:]\s*["\x27][A-Za-z0-9+/=]{20,}' --include="*.py" --include="*.js" --include="*.ts" --include="*.go" --include="*.rs" --include="*.env" . 2>/dev/null | grep -v 'example\|sample\|test\|mock\|placeholder' | head -3 || true)�[0m
 �[36;1mif [ -n "$SECRETS" ]; then�[0m
 �[36;1m  echo "::error::Potential hardcoded secrets detected — this FAILS the job:"�[0m

GitHub Actions: Governance / governance _ Security policy checks: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run set -uo pipefail
 �[36;1mset -uo pipefail�[0m
 �[36;1mDIR=.github/canonical-references�[0m
 �[36;1mif [ ! -d "$DIR" ]; then�[0m
 �[36;1m  echo "ℹ️  [R5] no $DIR/ — skipped (repo has not opted in)"�[0m
 �[36;1m  exit 0�[0m
 �[36;1mfi�[0m
 �[36;1mif ! command -v python3 >/dev/null 2>&1; then�[0m
 �[36;1m  echo "❌ [R5] python3 missing on runner — required for YAML rule parsing"�[0m
 �[36;1m  exit 2�[0m
 �[36;1mfi�[0m
 �[36;1mpython3 - <<'PY'�[0m
 �[36;1mimport os, sys, glob, subprocess�[0m
 �[36;1mtry:�[0m
 �[36;1m    import yaml�[0m
 �[36;1mexcept ImportError:�[0m
 �[36;1m    sys.exit("❌ [R5] PyYAML not installed on runner; install python3-yaml")�[0m
 �[36;1m�[0m
 �[36;1mdir_ = ".github/canonical-references"�[0m
 �[36;1mfiles = sorted(glob.glob(f"{dir_}/*.yml") + glob.glob(f"{dir_}/*.yaml"))�[0m
 �[36;1mif not files:�[0m
 �[36;1m    print(f"ℹ️  [R5] {dir_}/ has no .yml/.yaml rules — skipped")�[0m
 �[36;1m    sys.exit(0)�[0m
 �[36;1m�[0m
 �[36;1mtotal = 0�[0m
 �[36;1mfor rf in files:�[0m
 �[36;1m    with open(rf, encoding="utf-8") as fh:�[0m
 �[36;1m        cfg = yaml.safe_load(fh)�[0m
 �[36;1m    if not isinstance(cfg, dict):�[0m
 �[36;1m        print(f"❌ [R5] {rf}: top-level must be a mapping"); total += 1; continue�[0m
 �[36;1m    rid  = cfg.get("id", os.path.basename(rf))�[0m
 �[36;1m    desc = cfg.get("description", "")�[0m
 �[36;1m    pats = cfg.get("patterns") or []�[0m
 �[36;1m    canon = cfg.get("canonical_pointer", "")�[0m
 �[36;1m    scope = (cfg.get("scope") or {})�[0m
 �[36;1m    includes = scope.get("include") or []�[0m
 �[36;1m    if not pats or not includes:�[0m
 �[36;1m        print(f"❌ [R5:{rid}] missing patterns or scope.include in {rf}")�[0m
 �[36;1m        total += 1; continue�[0m
 �[36;1m    # exclude self-references�[0m
 �[36;1m    skip = set(["CHANGELOG.md", "CHANGELOG.adoc", rf])�[0m
 �[36;1m    if canon: skip.add(canon)�[0m
 �[36;1m    rule_hits = 0�[0m
 �[36;1m    for f_ in includes:�[0m
 �[36;1m        if f_ in skip or not os...

GitHub Actions: Governance / 7_governance _ Language _ package anti-pattern policy.txt: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run SCRIPT=".standards-checkout/scripts/check-ts-allowlist.sh"
 �[36;1mSCRIPT=".standards-checkout/scripts/check-ts-allowlist.sh"�[0m
 �[36;1mif [ ! -f "$SCRIPT" ] && [ "$GITHUB_REPOSITORY" = "hyperpolymath/standards" ] \�[0m
 �[36;1m   && [ -f scripts/check-ts-allowlist.sh ]; then�[0m
 �[36;1m  SCRIPT="scripts/check-ts-allowlist.sh"�[0m
 �[36;1m  echo "Using this repository's own copy (standards self-check)."�[0m
 �[36;1mfi�[0m
 �[36;1mif [ ! -f "$SCRIPT" ]; then�[0m
 �[36;1m  echo "::error::check-ts-allowlist gate not found in standards@main or locally"�[0m

GitHub Actions: Governance / governance _ Language _ package anti-pattern policy: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run SCRIPT=".standards-checkout/scripts/check-ts-allowlist.sh"
 �[36;1mSCRIPT=".standards-checkout/scripts/check-ts-allowlist.sh"�[0m
 �[36;1mif [ ! -f "$SCRIPT" ] && [ "$GITHUB_REPOSITORY" = "hyperpolymath/standards" ] \�[0m
 �[36;1m   && [ -f scripts/check-ts-allowlist.sh ]; then�[0m
 �[36;1m  SCRIPT="scripts/check-ts-allowlist.sh"�[0m
 �[36;1m  echo "Using this repository's own copy (standards self-check)."�[0m
 �[36;1mfi�[0m
 �[36;1mif [ ! -f "$SCRIPT" ]; then�[0m
 �[36;1m  echo "::error::check-ts-allowlist gate not found in standards@main or locally"�[0m

GitHub Actions: Governance / governance _ Language _ package anti-pattern policy: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run SCRIPT=".standards-checkout/tools/policy/check-language-policy.sh"
 �[36;1mSCRIPT=".standards-checkout/tools/policy/check-language-policy.sh"�[0m
 �[36;1mif [ ! -f "$SCRIPT" ] && [ -f tools/policy/check-language-policy.sh ]; then�[0m
 �[36;1m  SCRIPT="tools/policy/check-language-policy.sh"�[0m
 �[36;1m  echo "Using this repository's own copy (standards self-check)."�[0m
 �[36;1mfi�[0m
 �[36;1mif [ ! -f "$SCRIPT" ]; then�[0m
 �[36;1m  echo "::error::language-policy gate not found in standards@main or locally"�[0m

GitHub Actions: Governance / 9_governance _ Workflow security linter.txt: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run if [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then
 �[36;1mif [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then�[0m
 �[36;1m  SCRIPT="tools/policy/check-workflows-parse.sh"�[0m
 �[36;1m  echo "Using this repository's own copy (standards self-lint)."�[0m
 �[36;1melse�[0m
 �[36;1m  SCRIPT=".standards-dupkey/tools/policy/check-workflows-parse.sh"�[0m
 �[36;1mfi�[0m
 �[36;1mif [ ! -f "$SCRIPT" ]; then�[0m
 �[36;1m  echo "::error::workflow parser gate not found in the pinned Standards revision or locally"�[0m

GitHub Actions: Governance / governance _ Workflow security linter: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run if [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then
 �[36;1mif [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then�[0m
 �[36;1m  SCRIPT="tools/policy/check-workflows-parse.sh"�[0m
 �[36;1m  echo "Using this repository's own copy (standards self-lint)."�[0m
 �[36;1melse�[0m
 �[36;1m  SCRIPT=".standards-dupkey/tools/policy/check-workflows-parse.sh"�[0m
 �[36;1mfi�[0m
 �[36;1mif [ ! -f "$SCRIPT" ]; then�[0m
 �[36;1m  echo "::error::workflow parser gate not found in the pinned Standards revision or locally"�[0m

GitHub Actions: Governance / governance _ Workflow security linter: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run # GitHub Actions REJECTS a workflow with duplicate keys: the run is
 �[36;1m# GitHub Actions REJECTS a workflow with duplicate keys: the run is�[0m
 �[36;1m# `failure` with no jobs, no log and no check run. Nothing else here�[0m
 �[36;1m# can see it, because yaml.safe_load silently keeps the LAST�[0m
 �[36;1m# duplicate and reports success — so the file "parses" and every�[0m
 �[36;1m# other lint passes. Measured 2026-08-05: nine workflows in hypatia�[0m
 �[36;1m# were dead this way, including a CodeQL workflow with zero�[0m
 �[36;1m# successful runs in its entire lifetime.�[0m
 �[36;1mset -euo pipefail�[0m
 �[36;1m# Standards exercises its pull-request scripts; every consumer uses�[0m
 �[36;1m# the canonical scripts fetched from this workflow's immutable�[0m
 �[36;1m# Standards revision.�[0m
 �[36;1mif [ "$GITHUB_REPOSITORY" = hyperpolymath/standards ]; then�[0m
 �[36;1m  SCRIPT="scripts/check-workflow-duplicate-keys.sh"�[0m
 �[36;1m  echo "Using this repository's own copy (standards self-lint)."�[0m
 �[36;1melse�[0m
 �[36;1m  SCRIPT=".standards-dupkey/scripts/check-workflow-duplicate-keys.sh"�[0m
 �[36;1mfi�[0m
 �[36;1mif [ ! -f "$SCRIPT" ]; then�[0m
 �[36;1m  echo "::error::duplicate-key checker not found — neither fetched from" \�[0m

GitHub Actions: Governance / 14_governance _ Well-Known (RFC 9116 + RSR).txt: style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run SECTXT=""
 �[36;1mSECTXT=""�[0m
 �[36;1m[ -f ".well-known/security.txt" ] && SECTXT=".well-known/security.txt"�[0m
 �[36;1m[ -f "security.txt" ] && SECTXT="security.txt"�[0m
 �[36;1mif [ -z "$SECTXT" ]; then�[0m
 �[36;1m  echo "::warning::No security.txt found."�[0m
 �[36;1m  exit 0�[0m
 �[36;1mfi�[0m
 �[36;1mgrep -q "^Contact:" "$SECTXT" || { echo "::error::Missing Contact field"; exit 1; }�[0m

GitHub Actions: Governance / governance _ Well-Known (RFC 9116 + RSR): style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run SECTXT=""
 �[36;1mSECTXT=""�[0m
 �[36;1m[ -f ".well-known/security.txt" ] && SECTXT=".well-known/security.txt"�[0m
 �[36;1m[ -f "security.txt" ] && SECTXT="security.txt"�[0m
 �[36;1mif [ -z "$SECTXT" ]; then�[0m
 �[36;1m  echo "::warning::No security.txt found."�[0m
 �[36;1m  exit 0�[0m
 �[36;1mfi�[0m
 �[36;1mgrep -q "^Contact:" "$SECTXT" || { echo "::error::Missing Contact field"; exit 1; }�[0m

GitHub Actions: Governance / governance _ Well-Known (RFC 9116 + RSR): style(ncl): format Nickel files with nickel format

Conclusion: failure

View job details

##[group]Run MIXED=$(grep -rE 'src="http://|href="http://' --include="*.html" --include="*.htm" . 2>/dev/null | grep -vE 'localhost|127\.0\.0\.1|example\.com|lol/|node_modules/|third-party/|vendor/' | head -5 || true)
 �[36;1mMIXED=$(grep -rE 'src="http://|href="http://' --include="*.html" --include="*.htm" . 2>/dev/null | grep -vE 'localhost|127\.0\.0\.1|example\.com|lol/|node_modules/|third-party/|vendor/' | head -5 || true)�[0m
 �[36;1mif [ -n "$MIXED" ]; then�[0m
 �[36;1m  echo "::error::Mixed content (HTTP in HTML)"�[0m

🔇 Additional comments (11)

.machine_readable/contractiles/_base.ncl (1)

51-52: LGTM!

Also applies to: 55-55, 64-64, 66-67, 119-120

.machine_readable/contractiles/adjust/adjust.ncl (1)

18-36: LGTM!

Also applies to: 42-51, 57-64

.machine_readable/contractiles/bust/bust.ncl (1)

20-39: LGTM!

Also applies to: 45-56, 61-68

.machine_readable/contractiles/dust/dust.ncl (1)

20-39: LGTM!

Also applies to: 45-56, 61-68

.machine_readable/contractiles/intend/intend.ncl (1)

26-44: LGTM!

Also applies to: 50-66, 74-83

.machine_readable/contractiles/must/must.ncl (1)

20-38: LGTM!

Also applies to: 46-55, 60-66

.machine_readable/contractiles/trust/trust.ncl (1)

20-39: LGTM!

Also applies to: 45-54, 61-81, 86-93

.machine_readable/coaptation/coapt.ncl (1)

27-31: LGTM!

Also applies to: 50-88, 127-128

.machine_readable/coaptation/grades.ncl (1)

30-31: LGTM!

Also applies to: 34-35

.machine_readable/coaptation/witness-map.ncl (1)

28-28: LGTM!

Also applies to: 41-41, 58-58, 67-67, 75-75, 80-80, 85-85

.machine_readable/arrival-pack/arrival-pack.ncl (1)

23-23: LGTM!

📝 Summary

Summary by CodeRabbit

Refactor
- Reformatted machine-readable configuration and contract definitions for improved consistency and readability.
- Standardised layout, indentation, line breaks, comments, and record formatting across arrival, coaptation, witness-map, and contract configuration files.
- Preserved all existing values, validation rules, defaults, runtime behaviour, and generated output.

Walkthrough

The pull request reformats Nickel schemas, contract configurations, coaptation definitions, witness maps, and arrival-pack bindings. Declared values, interpolation fields, evaluation logic, witness content, and runtime behaviour remain unchanged.

Changes

Nickel formatting normalisation

Layer / File(s)	Summary
Contract schema and runner formatting `.machine_readable/contractiles/_base.ncl`, `.machine_readable/contractiles//.ncl`	Reformats shared schemas and contract configurations. Fields, defaults, controls, security settings, and runner behaviour remain unchanged.
Coaptation evaluation and witness formatting `.machine_readable/coaptation/coapt.ncl`, `.machine_readable/coaptation/grades.ncl`, `.machine_readable/coaptation/witness-map.ncl`	Reformats conditional expressions, evaluation records, grade mappings, witness arrays, and record layout. Evaluation logic and witness content remain unchanged.
Arrival-pack binding formatting `.machine_readable/arrival-pack/arrival-pack.ncl`	Condenses the bindings and provenance interpolation. The imported path, pin, and interpolated fields remain unchanged.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Refactor

Suggested reviewers: {{owner}}

Merge Risk: ⚪ Minimal · up to 2eba8

This change normalizes Nickel formatting without altering configuration values or runtime behavior, so it is ready to merge.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the main change: formatting Nickel files with the Nickel formatter.
Description check	✅ Passed	The description clearly explains the purpose, scope, excluded files, and verification results. It does not use the template headings or include the completed checklist, but the content is sufficiently…
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🛠️ Fix failing CI checks

Commit to this branch
Create a new PR

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

_{A rabbit reads each line,
The patch grows clear beneath the moon,
Small changes hop in place,
Tests guard the garden path,
Reviews bloom before the dawn.}

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

coderabbitai · 2026-09-22T13:45:03Z

✅ Coding Agent task started: View task and status

The task will inspect the CI failures, validate its fix, and commit the fix to this branch automatically.

Note: Fixing CI failures is a beta feature and may encounter errors. Expect some limitations and changes as we gather feedback and continue to improve it.

sonarqubecloud · 2026-09-22T14:33:19Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

hyperpolymath mentioned this pull request Sep 22, 2026

ci-pipeline.yml Nickel job globs *.k9.ncl and generated-input .ncl: 20 of 29 template files can never pass hyperpolymath/standards#982

Closed

5 tasks

coderabbitai Bot approved these changes Sep 22, 2026

View reviewed changes

Merge branch 'main' into fix/format-nickel-files

1f74a0a

hyperpolymath merged commit a0ec022 into main Sep 22, 2026
5 checks passed

hyperpolymath deleted the fix/format-nickel-files branch September 22, 2026 14:32

Navigation Menu

Uh oh!

style(ncl): format Nickel files with nickel format - #195

style(ncl): format Nickel files with nickel format#195
hyperpolymath merged 2 commits into
mainfrom
fix/format-nickel-files

hyperpolymath commented Sep 22, 2026

Uh oh!

coderabbitai Bot commented Sep 22, 2026 •

edited

Loading

Summary by CodeRabbit

Walkthrough

Changes

Uh oh!

coderabbitai Bot commented Sep 22, 2026 •

edited

Loading

Uh oh!

Uh oh!

sonarqubecloud Bot commented Sep 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

hyperpolymath commented Sep 22, 2026

Verification

Not an auto-fix

Uh oh!

coderabbitai Bot commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Walkthrough

Changes

Uh oh!

coderabbitai Bot commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

sonarqubecloud Bot commented Sep 22, 2026

Quality Gate passed

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

coderabbitai Bot commented Sep 22, 2026 •

edited

Loading

coderabbitai Bot commented Sep 22, 2026 •

edited

Loading