diff --git a/.gitea/workflows/doxygen-pages.yml b/.gitea/workflows/doxygen-pages.yml new file mode 100644 index 0000000..f4bb546 --- /dev/null +++ b/.gitea/workflows/doxygen-pages.yml @@ -0,0 +1,78 @@ +name: Doxygen → Codeberg Pages + +# Auto-publish Doxygen HTML to https://tmoussa.codeberg.page/ConformalLabpp/ +# every time the public API or docs source changes on main. +# +# Pattern: mirrors mirror-to-codeberg.yml — reuses the existing +# CODEBERG_TOKEN secret + HTTPS push. No new secret setup required. +# +# Trigger: push to main that touches code/include/**, Doxyfile, the +# filter script, doc/**/*.md, README.md, or this workflow file. Also +# manually triggerable via workflow_dispatch. + +on: + push: + branches: + - main + paths: + - "code/include/**" + - "Doxyfile" + - "scripts/doxygen-md-filter.sh" + - "doc/**/*.md" + - "README.md" + - "CLAUDE.md" + - ".gitea/workflows/doxygen-pages.yml" + workflow_dispatch: {} + +jobs: + publish: + runs-on: eulernest + container: + image: git.eulernest.eu/conformallab/ci-cpp:latest + + steps: + - uses: actions/checkout@v4 + + - name: Configure CMake (Doxygen target only — no compiler needed) + run: cmake -S code -B build + + - name: Build Doxygen HTML + run: | + cmake --build build --target doc + test -f doc/doxygen/html/index.html + warnings=$(wc -l < doc/doxygen/doxygen-warnings.log) + echo "DOC ▸ Doxygen warnings: $warnings" + if [ "$warnings" -gt 0 ]; then + echo "::warning::Doxygen produced $warnings warning(s) — review doc/doxygen/doxygen-warnings.log" + head -30 doc/doxygen/doxygen-warnings.log + fi + + - name: Publish HTML to codeberg pages branch + env: + CODEBERG_TOKEN: ${{ secrets.CODEBERG_TOKEN }} + run: | + set -eu + # Build the publish payload in a clean scratch dir so the + # orphan branch contains only the Doxygen output (and a + # marker README), never any build/source artefacts. + publish_dir=$(mktemp -d) + cp -r doc/doxygen/html/. "$publish_dir/" + + cat > "$publish_dir/README.txt" < (auto-published from `main` by `.gitea/workflows/doxygen-pages.yml`) — but symbols without explicit doc comments show only their signature. Public API surface (entry functions, named-parameter helpers, traits typedefs) has hand-written Doxygen; internal helpers vary. | clean (0 warnings) under current policy; not yet enforced "no undocumented symbol"; pursued on a separate branch | ~3 days to drive `WARN_IF_UNDOCUMENTED = YES` to zero | +| **`check-test-counts.sh` not wired into CI.** The script exists, runs locally, and gives the correct answer (23 + 234 = 257 / 0 skipped). CI does not yet fail on a mismatch. | local guard exists, CI integration pending next workflow touch | ~1 hour | +| **CP-Euclidean and Inversive-Distance research-track entries.** Both ship a working DCE solver, but lack the auxiliary utilities the Euclidean / HyperIdeal entries have (curvature inspection helpers, edge-flip Delaunay maintenance for ID). Out of port scope; listed in [`research-track.md`](../roadmap/research-track.md). | research-track, not blocking | per-utility | +| **`StereographicUnwrapper`, `CircleDomainUnwrapper`, `CuttingUtility`, `KoebePolyhedron`.** Mentioned in roadmap + research-track docs but not yet ported. Java versions still authoritative. | documented as Phase 11+ / optional | weeks each — explicit "out of port scope unless requested" | + +The honest framing for the meeting: **the porting layer hits its target +for the 5 Phase-8b-Lite DCE entries; the hackability layer is +demonstrably in place (3 tutorials, named-parameter chaining, +Doxygen-HTML); the research-track items are scoped but not built.** + +--- + ## Open questions for the external reviewer Items where the project would benefit from a second opinion: diff --git a/doc/release-policy.md b/doc/release-policy.md index 4a53633..e0a8472 100644 --- a/doc/release-policy.md +++ b/doc/release-policy.md @@ -82,7 +82,7 @@ rule is: | Number / string | Single source of truth | Anywhere else | |-----------------------------|--------------------------------|--------------------------------------| -| **Total / per-suite test counts** | `doc/api/tests.md` | Use qualitative phrasing + link, e.g. "full test suite passes, 0 skipped — see [`doc/api/tests.md`](doc/api/tests.md)". | +| **Total / per-suite test counts** | `doc/api/tests.md` | Use qualitative phrasing + link, e.g. "full test suite passes, 0 skipped — see [`doc/api/tests.md`](api/tests.md)". | | **Project version** | `CITATION.cff` `version:` field | Don't hardcode in headers, READMEs, doc bodies. Reference by name ("v0.9.0") only when the historical version actually matters (changelog entries, release-note bodies, Phase-milestone tables). | | **Release date** | `CITATION.cff` `date-released:` + `CHANGELOG.md` section header | Don't hardcode elsewhere. | | **Test-suite description per file** | `doc/api/tests.md` | Headers and code files may list the suites they implement, but never claim a count of suites elsewhere. | diff --git a/doc/tutorials/block-fd-hessian.md b/doc/tutorials/block-fd-hessian.md index 81778d5..7849f26 100644 --- a/doc/tutorials/block-fd-hessian.md +++ b/doc/tutorials/block-fd-hessian.md @@ -37,7 +37,7 @@ as `face_angles_from_local_dofs(...)`. > for circle patterns and Koebe's theorem.* Trans. AMS 356(2), 659–689. **Prerequisite:** familiarity with the hyper-ideal functional itself -(see [`doc/math/hyper-ideal.md`](../math/hyper-ideal.md)), and with the +(see [`doc/math/geometry-modes.md`](../math/geometry-modes.md)), and with the generic functional-porting pattern of [`doc/tutorials/add-inversive-distance.md`](add-inversive-distance.md). diff --git a/scripts/doxygen-md-filter.sh b/scripts/doxygen-md-filter.sh new file mode 100755 index 0000000..fc2fcfb --- /dev/null +++ b/scripts/doxygen-md-filter.sh @@ -0,0 +1,28 @@ +#!/bin/sh +# scripts/doxygen-md-filter.sh +# +# Doxygen FILTER for *.md files. Rewrites repository-relative markdown links +# (e.g. `[label](doc/api/tests.md)` or `[label](architecture/foo.md)`) into +# basename-only form (`[label](tests.md)`, `[label](foo.md)`). +# +# Why: GitHub's markdown renderer needs the full relative path to resolve a +# link, but Doxygen indexes every .md file in INPUT by basename and treats +# any "/" in a link target as an unresolvable \ref. Without this filter we +# get ~27 spurious warnings from README.md and CLAUDE.md. +# +# The filter is purely a Doxygen-side concern; the on-disk markdown is +# untouched, so GitHub rendering keeps working. +# +# Precondition: no two .md files in the project share a basename. Verified +# at the time of writing by: +# find . -name "*.md" -not -path "*/build*/*" -not -path "*/code/deps/*" \ +# -not -path "*/.git/*" | xargs -n1 basename | sort | uniq -c | awk '$1>1' +# (empty output ⇒ no collisions). +# +# Usage: configured via Doxyfile FILTER_PATTERNS = *.md=scripts/doxygen-md-filter.sh +set -eu +# Convert any markdown link target that points at a .md file into an HTML +# anchor. Doxygen renders HTML verbatim and skips its (broken) \ref +# resolution; the rendered Doxygen HTML still hyperlinks to the source .md. +# Pattern: [label](path/to/file.md) → label +sed -E 's#\[([^]]+)\]\(([a-zA-Z0-9_./-]+\.md)\)#\1#g' "$1"