stuff

[vinniefalco]

Summary by CodeRabbit

• Chores

  • Added CMakePresets.json configuration file for build system setup.

• Tests

  • Updated test suite to use blocking test runner across routing and route handling tests.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request refactors test execution patterns across the server module, replacing synchronous capy::run_sync() calls with a blocking test runner capy::test::run_blocking(), removes an unused include from source code, and introduces a CMakePresets.json configuration file with baseline CMake requirements.

Changes

Cohort / File(s)	Summary
CMake Configuration CMakePresets.json	New file defining CMake preset structure with version 6, minimum CMake requirement (3.25.0), and empty arrays for configure and build presets.
Source Code Cleanup src/server/route_handler.cpp	Removed include directive for <boost/capy/ex/run_sync.hpp> (no functional code changes).
Test Harness Refactoring test/unit/server/flat_router.cpp, test/unit/server/router.cpp	Updated include from boost/capy/ex/run_sync.hpp to boost/capy/test/run_blocking.hpp; replaced four call sites of capy::run_sync() with capy::test::run_blocking() in test functions (testCopyConstruction, testCopyAssignment, and router dispatch tests).
Test Infrastructure Update test/unit/server/route_handler.cpp	Added new struct any_write_buffers in namespace boost::http with std::array<capy::const_buffer, 16> member; updated include to boost/capy/test/run_blocking.hpp and added <array> header; refactored dispatch invocation to use blocking test runner with result capture via lambda.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A hop through headers, refactoring's sweet song,
From sync to blocking, the tests march along,
New presets configured, old paths now gone,
CMake and harnesses, together made strong!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 14.29% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'stuff' is vague and generic, using a non-descriptive term that conveys no meaningful information about the changeset.	Replace with a descriptive title that summarizes the main change, such as 'Replace capy::run_sync with capy::test::run_blocking in tests' or 'Add CMakePresets.json and migrate tests to blocking runner'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@CMakePresets.json`:
- Around line 1-9: CMakePresets.json declares "version": 6 and
cmakeMinimumRequired 3.25.0 which conflicts with the upper bound in
CMakeLists.txt (cmake_minimum_required(VERSION 3.8...3.20)); either make the
preset compatible with the project's allowed CMake range or relax the project's
CMake constraint. Fix by choosing one: (A) update CMakeLists.txt's
cmake_minimum_required range to include 3.25 (remove or raise the upper bound so
3.25+ is allowed), or (B) downgrade CMakePresets.json to a preset version
compatible with CMake 3.20 (change "version" and/or cmakeMinimumRequired to 3.20
or lower) so the preset and CMakeLists.txt agree.

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash

# Search for cmake_minimum_required in CMakeLists.txt
echo "=== CMakeLists.txt cmake_minimum_required ===" 
rg -n "cmake_minimum_required" --type cmake

# Search for cmake version references in CI workflows
echo -e "\n=== CI workflows (yml/yaml) with cmake references ==="
rg -n "cmake" -t yaml | head -50

# Search for cmake in README files
echo -e "\n=== README files mentioning cmake ==="
rg -n -i "cmake" -g "README*" | head -30

# Check if CMakePresets.json exists and confirm version
echo -e "\n=== CMakePresets.json details ==="
find . -name "CMakePresets.json" -type f | head -5

Repository: cppalliance/http

Length of output: 507

---

CMakePresets.json version 6 conflicts with CMakeLists.txt version constraints.

CMakePresets version 6 requires CMake 3.25+, but CMakeLists.txt specifies a maximum of 3.20 (cmake_minimum_required(VERSION 3.8...3.20)). Either update CMakeLists.txt to allow CMake 3.25+, or downgrade CMakePresets.json to a version compatible with CMake 3.20 or earlier.

🤖 Prompt for AI Agents

In `@CMakePresets.json` around lines 1 - 9, CMakePresets.json declares "version":
6 and cmakeMinimumRequired 3.25.0 which conflicts with the upper bound in
CMakeLists.txt (cmake_minimum_required(VERSION 3.8...3.20)); either make the
preset compatible with the project's allowed CMake range or relax the project's
CMake constraint. Fix by choosing one: (A) update CMakeLists.txt's
cmake_minimum_required range to include 3.25 (remove or raise the upper bound so
3.25+ is allowed), or (B) downgrade CMakePresets.json to a preset version
compatible with CMake 3.20 (change "version" and/or cmakeMinimumRequired to 3.20
or lower) so the preset and CMakeLists.txt agree.

[cppalliance-bot]

An automated preview of the documentation is available at https://252.http.prtest3.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-01-22 16:42:59 UTC