Skip to content

Add AI-ingestible framework brief#115

Open
Ulrond wants to merge 6 commits into
developfrom
feature/ai-brief
Open

Add AI-ingestible framework brief#115
Ulrond wants to merge 6 commits into
developfrom
feature/ai-brief

Conversation

@Ulrond
Copy link
Copy Markdown
Contributor

@Ulrond Ulrond commented Mar 13, 2026

Summary

  • Adds docs/ai-brief.md -- a comprehensive single-file brief capturing ut-control's KVP parser, WebSocket control plane, and logging APIs
  • Enables AI tools to understand the library without reading every source file
  • Covers all public C APIs with signatures, the build system, vendored dependencies, and integration with ut-core/ut-raft

Closes #114

Test plan

  • Verify brief accuracy against source headers
  • Confirm KVP API completeness against ut_kvp.h
  • Review control plane API against ut_control_plane.h

@Ulrond Ulrond requested a review from a team as a code owner March 13, 2026 13:24
Copilot AI review requested due to automatic review settings March 13, 2026 13:24
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an AI-ingestible, single-file technical brief for ut-control intended to summarize the KVP parser, WebSocket/HTTP control plane, logging APIs, build/dependency setup, and integration points (ut-core / ut-raft), so AI tools can understand the library without traversing the full codebase.

Changes:

  • Introduces docs/ai-brief.md documenting public C APIs for KVP, control plane, and logging.
  • Documents build system behavior (configure.sh, Makefile) and vendored dependency versions.
  • Describes repo directory layout and external integration model.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread docs/ai-brief.md Outdated
Comment thread docs/ai-brief.md Outdated
Comment thread docs/ai-brief.md Outdated
Comment thread docs/ai-brief.md Outdated
Comment thread docs/ai-brief.md Outdated
Comment thread docs/ai-brief.md Outdated
Comment thread docs/ai-brief.md
Comment thread docs/ai-brief.md
Comment thread docs/ai-brief.md Outdated
@Ulrond Ulrond self-assigned this May 15, 2026
@Ulrond Ulrond added the documentation Improvements or additions to documentation label May 15, 2026
kanjoe24
kanjoe24 previously approved these changes May 15, 2026
Copy link
Copy Markdown
Contributor

@kanjoe24 kanjoe24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

Copilot AI review requested due to automatic review settings May 15, 2026 16:15
…sage queue semantics, log colour stripping, OpenSSL/TLS build flags, test framework name, MAX_POST_DATA_SIZE limit, UT_KVP_STATUS_MAX, soften API-coverage claim
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.

Comment thread docs/ai-brief.md
Comment thread docs/ai-brief.md
Comment thread docs/ai-brief.md
Comment on lines +345 to +348

## 7. Integration with ut-raft (Python side)

ut-raft is the Python-based test orchestration framework. It communicates with
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May be this can be reworded:

send yaml/json as raw binary payloads

Comment thread docs/ai-brief.md
@Ulrond Ulrond requested a review from kanjoe24 May 17, 2026 07:29
Comment thread docs/ai-brief.md
- `simple_websocket.py` -- minimal WebSocket client
- `curl-client-yaml.sh` -- sends YAML via HTTP POST
- `curl-client-json.sh` -- sends JSON via HTTP POST
- `curl-client-binary.sh` -- sends binary data via HTTP POST
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May be this can be reworded:

send yaml/json as raw binary payloads via HTTP POST

Copy link
Copy Markdown
Contributor

@kanjoe24 kanjoe24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment added.

Rest looks fine.

Copilot AI review requested due to automatic review settings May 19, 2026 15:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread docs/ai-brief.md
| **asprintf** | 0.0.3 | Portable `asprintf()` (compiled into libut_control) |
| **libwebsockets** | 4.3.3 | WebSocket/HTTP server (static `.a` linked in) |
| **curl** | 8.8.0 | HTTP client for `!include` URL resolution (static or system) |
| **OpenSSL** | 1.1.1w | Crypto dependency (static or system). Note: the default build disables TLS -- libwebsockets is built with `-DLWS_WITH_SSL=OFF` and curl with `-DCMAKE_USE_OPENSSL=OFF`, so `wss://`/HTTPS are not enabled. |
Comment thread docs/ai-brief.md
- `simple_websocket.py` -- minimal WebSocket client
- `curl-client-yaml.sh` -- sends YAML via HTTP POST
- `curl-client-json.sh` -- sends JSON via HTTP POST
- `curl-client-binary.sh` -- sends binary data via HTTP POST
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add AI-ingestible framework brief

3 participants