ma-agents 3.4.7 → 3.4.8

package/.roo/skills/cpp-const-correctness/SKILL.md

@@ -1,63 +0,0 @@
----
-name: C++ Const-Correctness & Logic
-description: Enforce immutability-by-default and push logic to compile-time using constexpr (C++14+).
----
-# C++ Const-Correctness & Logic (Core Guidelines Section Con & ES)
-
-This skill ensures that C++ code is "immutable by default," reducing side effects and enabling better compiler optimizations.
-
-## Policies
-
-### 1. By Default, Make Objects Immutable
-*   **Rule**: Declare all variables `const` unless they *must* be modified.
-*   **Action**: 
-    - Use `const auto x = ...;` instead of `auto x = ...;`.
-    - Use `const` for function results that won't be modified.
-*   **Rationale**: Prevents accidental state mutation and improves code reasoning.
-
-### 2. Const Member Functions
-*   **Rule**: Any member function that does not modify the object's observable state must be `const`.
-*   **Action**: Append `const` to the function signature: `T getValue() const { ... }`.
-*   **Rationale**: Necessary for calling functions on `const` objects and `const&` parameters.
-
-### 3. Compute at Compile-Time (`constexpr`)
-*   **Rule**: Prefer `constexpr` for any value or simple logic that *can* be computed at compile-time.
-*   **Action**:
-    - Use `constexpr` for constants.
-    - Declare mathematical or configuration functions as `constexpr` (C++14 allows complex logic in `constexpr`).
-*   **Rationale**: Moves work from runtime to compile-time, reducing binary size and execution time.
-
-### 4. Consistent Parameter Logic
-*   **Rule**: Avoid passing by value for large types; avoid passing by non-const reference unless it's an "out" or "in-out" parameter.
-*   **Action**:
-    - **In parameters**: Pass cheap types (int, double, pointers, `string_view`) by value. Pass large types by `const&`.
-    - **Out/In-out parameters**: Pass by `T&`.
-    - **Sinks**: Pass by `T&&` (rvalue reference).
-
-## Examples
-
-### Before (Mutable/Runtime Logic)
-```cpp
-int getMultiplier(int x) {
-    return x * 10;
-}
-
-void process() {
-    auto data = loadData();
-    auto mult = getMultiplier(5); // mutable by accident
-    data.apply(mult);
-}
-```
-
-### After (Const/Compile-time Logic)
-```cpp
-constexpr int getMultiplier(int x) {
-    return x * 10;
-}
-
-void process() {
-    const auto data = loadData(); // data won't change
-    constexpr auto mult = getMultiplier(5); // computed at compile time
-    data.apply(mult);
-}
-```

package/.roo/skills/cpp-const-correctness/examples/const_correctness.md

@@ -1,54 +0,0 @@
-# Const-Correctness Examples (C++14+)
-
-### 1. Complex Const Initialization (Lambda)
-Sometimes you need to conditionally initialize a variable, but you want it to be `const` afterward.
-
-**Modern (C++14):**
-```cpp
-const auto path = []() {
-    if (auto env = std::getenv("APP_PATH")) {
-        return std::string(env);
-    }
-    return std::string("/default/path");
-}(); // Immediately invoked lambda
-```
-
-### 2. Const-Correct Member Functions
-Ensure getters and inspectors are `const`.
-
-```cpp
-class Config {
-    int timeout = 30;
-public:
-    // This is an inspector, must be const
-    int getTimeout() const { return timeout; }
-    
-    // This is a mutator, cannot be const
-    void setTimeout(int t) { timeout = t; }
-};
-```
-
-### 3. C++14 constexpr Logic
-C++14 expanded `constexpr` significantly (loops, branches allowed).
-
-```cpp
-constexpr int fibonacci(int n) {
-    if (n <= 1) return n;
-    int a = 0, b = 1;
-    for (int i = 2; i <= n; ++i) {
-        int next = a + b;
-        a = b;
-        b = next;
-    }
-    return b;
-}
-
-// Result is computed by the compiler
-constexpr int fib10 = fibonacci(10);
-```
-
-### 4. Parameter Passing Guide
-- `void f(int)` : Fast to copy (Value)
-- `void f(const LargeObject&)` : Expensive to copy (Const Ref)
-- `void f(LargeObject&)` : I will modify this (Ref)
-- `void f(LargeObject&&)` : I will MOVE/OWN this (Rvalue Ref)

package/.roo/skills/cpp-memory-handling/SKILL.md

@@ -1,42 +0,0 @@
----
-name: C++ Safe Memory Handling
-description: Enforces Modern C++ practices (RAII, Smart Pointers) to prevent memory leaks, dangling pointers, and buffer overflows.
----
-# C++ Safe Memory Handling
-
-Guidelines for writing robust, memory-safe C++ code by leveraging Modern C++ features and RAII principles.
-
-## Mandatory Policies
-
-### 1. Smart Pointers by Default
-You MUST use smart pointers for all heap allocations.
-- Use `std::unique_ptr` for exclusive ownership. This is the preferred default.
-- Use `std::shared_ptr` only when multiple objects share ownership of a resource.
-- Use `std::weak_ptr` to break circular dependencies or for non-owning references that must be checked for validity.
-
-### 2. Forbid raw new/delete
-You MUST NOT use raw `new` and `delete` operators.
-- Use `std::make_unique` (C++14+) or `std::make_shared` to allocate objects on the heap.
-- Exceptions are only allowed within low-level custom container implementations, which must be encapsulated.
-
-### 3. Prefer Value Semantics
-Always prefer stack-allocated objects and value semantics over pointer-based indirection unless heap allocation is strictly necessary (e.g., polymorphism, very large objects, or dynamic lifetimes).
-
-### 4. RAII for All Resources
-Every resource (file handles, sockets, mutexes, memory) MUST be managed by an object whose constructor acquires the resource and destructor releases it. Never rely on manual cleanup blocks.
-
-### 5. Standard Containers
-NEVER use raw buffers or manually managed arrays.
-- Use `std::vector` for dynamic arrays.
-- Use `std::string` for text.
-- Use `std::array` (C++11) for fixed-size stack arrays.
-- Access elements using `.at()` if bounds checking is required for security-critical logic.
-
-## Critical Rules
-- **No Dangling Pointers**: Never return a pointer or reference to a local stack-allocated variable.
-- **Rule of Zero/Five**: Follow the "Rule of Zero" (prefer components that handle their own resource management). If manual management is needed, correctly implement the "Rule of Five".
-- **Nullable Checks**: Always check pointers (even smart ones) for `nullptr` before dereferencing if there is any chance they could be empty.
-
-## Resources
-- [Modern C++ Examples](file:///skills/cpp-memory-handling/examples/modern-cpp.md)
-- [Smart Pointer Reference](file:///skills/cpp-memory-handling/examples/smart-pointers.md)

package/.roo/skills/cpp-memory-handling/examples/modern-cpp.md

@@ -1,49 +0,0 @@
-# Modern C++ Safe Memory Examples
-
-Demonstrating RAII and safe resource management in modern C++.
-
-## RAII Resource Management
-
-```cpp
-#include <iostream>
-#include <fstream>
-#include <vector>
-#include <memory>
-
-class FileHandler {
-public:
-    FileHandler(const std::string& filename) : file(filename) {
-        if (!file.is_open()) throw std::runtime_error("Could not open file");
-    }
-    // Destructor automatically closes the file
-    ~FileHandler() { if (file.is_open()) file.close(); }
-
-    void write(const std::string& data) { file << data; }
-
-private:
-    std::ofstream file;
-};
-
-void log_data(const std::string& message) {
-    // RAII handles lifetime. No manual close needed.
-    FileHandler handler("app.log");
-    handler.write(message);
-}
-```
-
-## Vector vs Raw Arrays
-
-```cpp
-// BAD: Manual memory management
-void bad_array() {
-    int* arr = new int[100];
-    // ... logic ...
-    delete[] arr; // Manual cleanup is error-prone
-}
-
-// GOOD: Automatic memory management
-void safe_vector() {
-    std::vector<int> arr(100);
-    // Vector manages its own heap memory. RAII cleans up at scope exit.
-}
-```

package/.roo/skills/cpp-memory-handling/examples/smart-pointers.md

@@ -1,46 +0,0 @@
-# C++ Smart Pointer Reference
-
-Guidelines for selecting and using smart pointers correctly.
-
-## unique_ptr (Preferred Default)
-- Represents **exclusive ownership**.
-- Zero runtime overhead compared to raw pointers.
-
-```cpp
-#include <memory>
-
-class Widget { /* ... */ };
-
-void exclusive_use() {
-    // Use make_unique for safety and efficiency
-    auto widget = std::make_unique<Widget>();
-    
-    // Ownership can be moved but not copied
-    std::unique_ptr<Widget> other = std::move(widget);
-}
-```
-
-## shared_ptr (For Shared Ownership)
-- Use only when **multiple** owners truly exist.
-- Reference counting has small overhead.
-
-```cpp
-void shared_use() {
-    auto shared_widget = std::make_shared<Widget>();
-    
-    auto owner1 = shared_widget; // Count = 2
-    auto owner2 = shared_widget; // Count = 3
-}
-```
-
-## weak_ptr (Breaking Cycles)
-- Non-owning reference. Must be locked before use.
-
-```cpp
-void weak_use(std::weak_ptr<Widget> weak_widget) {
-    if (auto shared = weak_widget.lock()) {
-        // Resource is still alive
-        shared->do_something();
-    }
-}
-```

package/.roo/skills/cpp-modern-composition/SKILL.md

@@ -1,64 +0,0 @@
----
-name: C++ Modern Composition
-description: Replace legacy C patterns with STL, Ranges, and modern C++ abstractions (C++14+).
----
-# C++ Modern Composition (Core Guidelines Section STL, CPL, ES)
-
-This skill forces the transition from legacy "C-style" C++ to modern, declarative, and safer C++ idioms.
-
-## Policies
-
-### 1. STL Algorithms over Manual Loops
-*   **Rule**: Never write a raw `for` or `while` loop for operations supported by `<algorithm>` or `<numeric>`.
-*   **Action**: 
-    - Use `std::any_of`, `std::all_of`, `std::find_if`, `std::transform`, `std::accumulate`.
-    - In C++20 projects, prefer `std::ranges` versions.
-*   **Rationale**: Reduces "off-by-one" errors and improves intent readability.
-
-### 2. Eradicate C-style Casts and Macros
-*   **Rule**: Ban `(type)value` and `#define` for logic/constants.
-*   **Action**:
-    - Use `static_cast`, `reinterpret_cast`, or `const_cast`.
-    - Replace macros with `constexpr` variables or `inline` functions/templates.
-*   **Rationale**: Named casts are searchable and safer; `constexpr` respects namespaces and scopes.
-
-### 3. Universal Initialization
-*   **Rule**: Prefer `{}` initialization to avoid the "Most Vexing Parse".
-*   **Action**: Use `auto x = Type{args...};` or `Type x{args...};`.
-*   **Rationale**: Prevents narrowing conversions (e.g., `int x{3.5}` is a compiler error, while `int x = 3.5` is not).
-
-### 4. Zero Raw Memory Manipulation
-*   **Rule**: Ban `memset`, `memcpy`, and `malloc`.
-*   **Action**:
-    - Use `std::fill`, `std::copy`, or `container.assign()`.
-    - Use `std::vector` or `std::array` instead of raw arrays.
-*   **Rationale**: Standard containers are bounds-checked (via `.at()`) and manage their own memory.
-
-## Examples
-
-### Before (C-style C++)
-```cpp
-#define MAX_BUFFER 1024
-int arr[MAX_BUFFER];
-memset(arr, 0, sizeof(arr));
-
-bool found = false;
-for (int i = 0; i < MAX_BUFFER; ++i) {
-    if (arr[i] == (int)someFloat) {
-        found = true;
-        break;
-    }
-}
-```
-
-### After (Modern C++)
-```cpp
-constexpr size_t MaxBuffer = 1024;
-std::array<int, MaxBuffer> arr;
-arr.fill(0);
-
-auto target = static_cast<int>(someFloat);
-bool found = std::any_of(arr.begin(), arr.end(), [target](int v) {
-    return v == target;
-});
-```

package/.roo/skills/cpp-modern-composition/examples/composition.md

@@ -1,51 +0,0 @@
-# Modern Composition Examples (C++14+)
-
-### 1. Data Transformation (Transform)
-**Legacy:**
-```cpp
-std::vector<int> inputs = {1, 2, 3};
-std::vector<int> outputs;
-for (size_t i = 0; i < inputs.size(); ++i) {
-    outputs.push_back(inputs[i] * 2);
-}
-```
-
-**Modern (C++14):**
-```cpp
-std::vector<int> inputs = {1, 2, 3};
-std::vector<int> outputs;
-outputs.reserve(inputs.size());
-std::transform(inputs.begin(), inputs.end(), std::back_inserter(outputs), 
-               [](int n) { return n * 2; });
-```
-
-### 2. Summation and Reduction (Accumulate)
-**Modern (C++14):**
-```cpp
-#include <numeric>
-
-std::vector<double> prices = {10.5, 20.0, 5.25};
-double total = std::accumulate(prices.begin(), prices.end(), 0.0);
-```
-
-### 3. Safe Narrowing Protection
-**Logic:**
-```cpp
-double d = 7.9;
-// int i{d}; // Error: narrowing conversion from 'double' to 'int'
-int i = static_cast<int>(d); // Explicit and safe
-```
-
-### 4. Replacing Function Macros
-**Legacy:**
-```cpp
-#define SQUARE(x) ((x)*(x))
-```
-
-**Modern (C++14):**
-```cpp
-template<typename T>
-constexpr T square(T t) {
-    return t * t;
-}
-```

package/.roo/skills/cpp-robust-interfaces/SKILL.md

@@ -1,55 +0,0 @@
----
-name: C++ Robust Interfaces
-description: Enforce contract-based design and strong typing in C++ interfaces (C++14+).
----
-# C++ Robust Interfaces (Core Guidelines Section I & E)
-
-This skill ensures that C++ interfaces follow contract-based design principles to improve safety, clarity, and maintainability.
-
-## Policies
-
-### 1. Make Interfaces Explicit
-*   **Rule**: Functions must explicitly state their requirements (pre-conditions) and guarantees (post-conditions).
-*   **Action**: Use `Expects()` and `Ensures()` (from GSL) or standardized comments if GSL is unavailable.
-*   **Rationale**: Detects integration bugs at the call site rather than deep in the implementation.
-
-### 2. Strong Typing over Primitive Types
-*   **Rule**: Avoid "Boolean Blindness" and `void*`.
-*   **Action**: 
-    - Use `enum class` for options instead of `bool`.
-    - Use `std::chrono` for time instead of `int`.
-    - Use specific types/structs instead of a long list of primitive arguments.
-
-### 3. Clear Ownership and Lifetime
-*   **Rule**: A raw pointer (`T*`) in an interface **must never** represent ownership.
-*   **Action**: 
-    - Use `gsl::not_null<T*>` for pointers that cannot be null.
-    - Use `std::unique_ptr` or `std::shared_ptr` ONLY if the function is participating in ownership/lifetime management.
-    - Prefer `T&` for arguments that must exist and aren't owned by the callee.
-
-### 4. Sequence Safety
-*   **Rule**: Never pass an array as a raw pointer + size.
-*   **Action**:
-    - Use `gsl::span<T>` (C++14/17 via GSL) or `std::span<T>` (C++20).
-    - Use `std::string_view` for read-only strings.
-
-## Examples
-
-### Before (Fragile Interface)
-```cpp
-// What does 'flag' mean? Is data allowed to be null? Is it an array?
-void processData(char* data, int size, bool flag);
-```
-
-### After (Robust Interface)
-```cpp
-#include <gsl/gsl>
-
-enum class ProcessingMode { Fast, Thorough };
-
-// Explicit contract: data must not be null, size is handled by span
-void processData(gsl::span<const char> data, ProcessingMode mode) {
-    Expects(!data.empty());
-    // ...
-}
-```

package/.roo/skills/cpp-robust-interfaces/examples/interfaces.md

@@ -1,56 +0,0 @@
-# Robust Interface Examples (C++14+)
-
-### 1. Avoiding Boolean Blindness
-**Incorrect:**
-```cpp
-void setWindowVisible(bool visible, bool animate);
-// Call site: setWindowVisible(true, false); // What is false?
-```
-
-**Correct (C++14):**
-```cpp
-enum class Visibility { Visible, Hidden };
-enum class Animation { Enabled, Disabled };
-
-void setWindowVisible(Visibility v, Animation a);
-// Call site: setWindowVisible(Visibility::Visible, Animation::Disabled);
-```
-
-### 2. Contract-Based Validation
-Using GSL (Guidelines Support Library) style:
-
-```cpp
-#include <gsl/gsl>
-
-class User {
-public:
-    // Ensure name is never null or empty
-    void setName(gsl::not_null<const char*> name) {
-        Expects(std::strlen(name) > 0);
-        this->name = name;
-    }
-private:
-    std::string name;
-};
-```
-
-### 3. Safe Sequences with Span
-Replaces pointer + length pairs which are prone to buffer overflows.
-
-```cpp
-#include <gsl/gsl> // Use gsl::span in C++14
-
-float calculateAverage(gsl::span<const float> values) {
-    Expects(!values.empty());
-    
-    float sum = 0.0f;
-    for (float v : values) {
-        sum += v;
-    }
-    return sum / values.size();
-}
-
-// Usage:
-// float arr[] = {1, 2, 3};
-// calculateAverage(arr); // Automatic conversion to span
-```