Skip to content

Add dedicated aicpp devcontainer, CI, and release packaging#10

Merged
drzo merged 3 commits into
mainfrom
copilot/create-devcontainer-for-aicpp
May 8, 2026
Merged

Add dedicated aicpp devcontainer, CI, and release packaging#10
drzo merged 3 commits into
mainfrom
copilot/create-devcontainer-for-aicpp

Conversation

Copilot AI commented May 8, 2026

Copy link
Copy Markdown

This addresses the missing developer environment and automation around the cpp/ implementation. It adds a ready-to-use devcontainer for aicpp bootstrapping and a dedicated GitHub Actions path for C++ build/test and release artifact packaging.

  • Dev container

    • Adds a dedicated .devcontainer for the cpp/ implementation
    • Installs the required native toolchain and system deps (readline, ninja, doxygen)
    • Preconfigures CMake to bootstrap cpp/build on container creation
  • aicpp CI

    • Adds a separate aicpp workflow instead of coupling C++ builds to the existing Rust CI
    • Builds and tests cpp/ on Ubuntu and macOS
    • Limits workflow token permissions to read-only for the CI path
  • aicpp release packaging

    • Adds a dedicated release workflow for the C++ implementation
    • Rebuilds and tests before packaging
    • Produces install-tree archives from cmake --install and publishes them on tagged releases
  • Documentation

    • Updates cpp/README.md and cpp/QUICKSTART.md to surface the devcontainer and automation entry points
    • Links the C/C++ implementation guide from the root README.md

Example of the added bootstrap flow:

{
  "name": "aichat-aicpp",
  "postCreateCommand": "cmake -S cpp -B cpp/build -G Ninja -DCMAKE_BUILD_TYPE=RelWithDebInfo -DBUILD_TESTS=ON -DBUILD_EXAMPLES=OFF"
}

Copilot AI assigned Copilot and drzo May 8, 2026
Copilot AI linked an issue May 8, 2026 that may be closed by this pull request
Comment thread .github/workflows/aicpp-ci.yaml Fixed
Copilot AI changed the title [WIP] Add devcontainer for aicpp boot and installation Add dedicated aicpp devcontainer, CI, and release packaging May 8, 2026
Copilot finished work on behalf of drzo May 8, 2026 08:02
Copilot AI requested a review from drzo May 8, 2026 08:02

@drzo drzo 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.

cooool

@drzo drzo marked this pull request as ready for review May 8, 2026 08:08
@drzo drzo merged commit a56ca1b into main May 8, 2026
6 of 9 checks passed
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.

implement aicpp

3 participants