Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

---

## [Unreleased] - v1.100.3a Repo hygiene Phase A slice 1a (H-10)

Smallest possible doc-only fix from the repo hygiene audit. Closes audit finding **H-10**: broken `[HEADER_SPEC.md]` link in `CONTRIBUTING.md:242` (file does not exist at repo root) and matching dangling reference in `tools/validate-headers.sh`.

### Changed

- `CONTRIBUTING.md`: section heading retitled from "HEADER_SPEC.md (File Headers)" to "File Headers". Removed the broken `[HEADER_SPEC.md](HEADER_SPEC.md)` link. Added a sentence noting that the inline section is itself the authoritative spec and that CI enforces it via `tools/validate-headers.sh`.
- `tools/validate-headers.sh`: error message no longer references the non-existent `HEADER_SPEC.md`. Pointer text now says "CONTRIBUTING.md, section 'File Headers' (authoritative spec)". Header comment updated to match.

### Out of scope (deferred to slice 1b)

- H-01 / H-02 / H-03 — dev-path cleanup. Separate micro-PR.

Lifecycle completion lane (PR-25..PR-30) remains explicitly **OPEN**.

---

## [Unreleased] - v1.100.2 SF-1 health CLI fix (released-host case)

Post-soak correctness fix derived directly from the PR-24 7-day passive soak. Side finding **SF-1** was logged on Day 6 (2026-04-26) on the released/no-tables host (lab4): `nftban health` printed a spurious `ERROR: Script failed` block on top of its own DOWN-status output.
Expand Down
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ Any AI (Claude, etc.) must obey:

All contributions **must** comply with these authoritative standards:

### 1. HEADER_SPEC.md (File Headers)
### 1. File Headers

Every source file must have a compliant header with:
- **SPDX-License-Identifier: MPL-2.0** (exactly one per file)
Expand All @@ -239,7 +239,7 @@ Every source file must have a compliant header with:
meta:inventory.privileges=""
```

See [HEADER_SPEC.md](HEADER_SPEC.md) for complete specification.
This section is the authoritative header spec. CI enforces it via `tools/validate-headers.sh`.

### 2. Coding Standards

Expand Down
7 changes: 4 additions & 3 deletions tools/validate-headers.sh
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@
set -Eeuo pipefail

# Validate SPDX + meta quoting + required inventory keys + bash set flags.
# Fails commit if any tracked file violates HEADER_SPEC.
# Fails commit if any tracked file violates the file-header spec
# defined in CONTRIBUTING.md section 'File Headers'.

repo_root="$(git rev-parse --show-toplevel)"
cd "$repo_root"
Expand Down Expand Up @@ -170,11 +171,11 @@ done
if [[ "$fail" -ne 0 ]]; then
echo
echo "=========================================="
echo "Commit blocked: HEADER_SPEC / coding standards validation failed."
echo "Commit blocked: header / coding standards validation failed."
echo "Fix headers/metadata or inventory lines, then re-stage and commit."
echo
echo "References:"
echo " - HEADER_SPEC.md"
echo " - CONTRIBUTING.md, section 'File Headers' (authoritative spec)"
echo " - https://nftban.com/coding-standards.html"
echo "=========================================="
exit 1
Expand Down
Loading