-
Notifications
You must be signed in to change notification settings - Fork 3
Build 932: V12_001/V12_002 Rename + Offline IPC Fix #4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
d66fcff
Build 930.1: Fix signed-delta math for Short fleet cascade cancel
mkalhitti-cloud f21ad70
Build 931: Hardened & Simplified Baseline — spurious flat guard, REAP…
mkalhitti-cloud 85ff3dc
Build 932: Final Project-Wide Architectural Polish
mkalhitti-cloud 99f49be
Build 932: Final Phase 8 Stabilization - Mission Certified Gold.
mkalhitti-cloud c17a83a
Build 932: Final Renaming Migration to V12_002 - Identity Shift Compl…
mkalhitti-cloud 5067cfd
Build 932: Rename V12_001/V12_002 + Offline IPC fix
mkalhitti-cloud File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| # Zero-Waste Discovery Protocol (V12.932) | ||
|
|
||
| ## 🛡️ MISSION | ||
| To prevent AI agents from wasting tokens, time, and "Context Space" on irrelevant legacy code, backups, and non-active logic nodes. | ||
|
|
||
| ## 1. Automated Exclusion Rules | ||
| AI agents MUST automatically exclude the following patterns from all global scans, refactoring tasks, and audits (unless explicitly instructed by the Director): | ||
|
|
||
| * **Backups**: `*.bak`, `*.bak_*`, `*.backup`, `*.old` | ||
| * **Datestamps**: Files containing `YYYYMMDD` patterns (e.g. `*.20260224.cs`) | ||
| * **Archives**: Files in `ARCHIVE_*`, `LEGACY_*`, or `DEVELOPMENT/TRASH` folders. | ||
| * **Binaries**: `*.exe`, `*.dll`, `*.pdb`, `*.user`, `*.suo` | ||
| * **Meta**: `.git`, `.qodo`, `.testsprite`, `.claude/cache` | ||
|
|
||
| ## 2. Active Fleet Focus | ||
| When a "Global Scan" or "Project Simplification" is requested, the agent MUST restrict its search space to the **Active Logic Fleet**: | ||
| * Directory: `src/` | ||
| * Inclusion: `*.cs` | ||
| * Exclusion: (All patterns in Section 1) | ||
|
|
||
| ## 3. Context Conservation | ||
| * **Single-File Priority**: If a bug is in `SIMA.cs`, do NOT read `Properties.cs` or `Entries.cs`. | ||
| * **Grep-First**: Always use `grep` to find specific line numbers before reading a full file. | ||
| * **No Redundant Scanning**: If a file was audited in the last 24 hours and the hash hasn't changed, do not re-scan it. | ||
|
|
||
| ## 4. Director Escalation | ||
| If an agent believes a legacy or backup file contains critical "Logic DNA" needed for a repair, it MUST ask the Director: *"I have found a potential fix in a legacy backup [filename]. Permission to ingest?"* | ||
|
|
||
| --- | ||
| > [!IMPORTANT] | ||
| > Failure to follow this protocol results in "Context Debt" and degrades the quality of AI-driven repairs. Protect the Token Budget. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,47 @@ | ||
| # Universal Standards Manifesto: V12 Permanent DNA | ||
|
|
||
| This document provides the immutable technical standards for all AI agents (Anthropic, Codex, Antigravity, Cursor, Gemini, Rovo Dev) working on the **Universal OR Strategy V12**. | ||
|
|
||
| ## 1. Zero-Trust Safety Protocol (The "Law") | ||
| * **Verify, Don't Assume:** Never assume the strategy state (e.g., `activePositions`) matches the broker state. Always use `FirstOrDefault` when looking up positions in `acct.Positions` using the instrument object identity. | ||
| * **Terminal Removal:** Only remove order references from dictionaries (`stopOrders`, `target1Orders`, etc.) after a broker-confirmed terminal state (`Filled`, `Cancelled`, `Rejected`, `Unknown`). | ||
| * **Lifecycle Guards:** Any manual flatten or entry must check `isFlattenRunning` to prevent ordering loops. | ||
|
|
||
| ## 2. Concurrency & Locking | ||
| * **StateLock Rule:** Every mutation of `activePositions`, `entryOrders`, `stopOrders`, or `expectedPositions` MUST occur inside a `lock(stateLock)`. | ||
| * **Volatile Optimization:** If a variable is marked `volatile` and is being used for a simple "read-only" check (e.g., `if (pos.RemainingContracts > 0)`), remove unnecessary locks to minimize thread contention. | ||
| * **Semaphore Guards:** All `_simaToggleSem.Wait()` calls must be paired with a `Release()` inside a `finally` block. | ||
|
|
||
| ## 3. Coding Style & Modularity | ||
| * **Modular Parity:** Maintain the split file structure (e.g., `SIMA.cs`, `Orders.Management.cs`). Use `partial class` correctly. | ||
| * **Naming:** Use `PascalCase` for Methods/Properties, `camelCase` for fields/locals. | ||
| * **Metabolic Elegance:** Prioritize readable, surgical logic over dense one-liners. | ||
|
|
||
| ## 4. Multi-Agent Collaboration Protocol | ||
| * **Advisor (Antigravity):** The **"General Manager."** Handles high-level brainstorming, diagnosing market issues, and engineering the core "Mission Prompt." | ||
| * **Desk Supervisor & Lab (Gemini CLI):** The **"Quant & Compliance."** Uses **Conductor/ODIN** to turn Antigravity prompts into rigid technical plans. Uses the **Sandbox** to test math/logic in Python before implementation. Runs local security/lint audits. | ||
| * **Lead Engineer (Claude Code):** The **"Execution Specialist."** Performs high-speed implementing with **Simultaneous Auditing**. | ||
| * **Forensic Auditor (Codex):** The **"Deep Logic Inspector."** Performs a 360-degree forensic scan for logic traps/leaks. | ||
| * **Secondary Auditor (Cursor AI):** The **"Peer Reviewer."** Performed by the Cursor agent to provide a cross-verification audit from a different model perspective. | ||
| * **Maintenance Inspector (Human):** The **"Final Sign-off."** Used by the Fund Manager in Cursor to perform visual review and manual polish. | ||
| * **Context Layer (OneContext):** The **"Order Book."** Maintains a shared trajectory of all agent thoughts and actions. | ||
| * **The Bridge:** Handoffs are managed via `implementation_plan.md`, `session_handoff.md`, and **OneContext snapshots**. | ||
|
|
||
| ## 5. Multi-Agent Parity & Sync Protocol | ||
| * **Unified Tooling (MCP):** All MCP servers configured for Claude (`.mcp.json`) must be mirrored in the Gemini CLI (`settings.json`). This ensures that if Claude has access to things like `csharp-lsp` or `tavily-search`, Gemini and Rovo Dev do too. | ||
| * **High-Performance Tooling (The Triple Threat):** | ||
| - **Tool Search (Discovery):** Used to find specialized tools on-demand. Prevents token-leak. | ||
| - **Fine-Grained Streaming (Data Flow):** Streams large parameters (e.g., mass-refactors) without buffering. Reduces initial latency. | ||
| - **PTC (Execution):** Runs discovered tools at high frequency inside code containers. Reduces round-trips. | ||
| * **Context Parity:** The Project DNA in `CLAUDE.md` must be identical to `GEMINI.md`. All agents must read from the same baseline instructions. | ||
| * **Environment Sync:** All agents (including Rovo Dev) must have access to the same local environment variables (API keys, project paths) to ensure execution parity. | ||
|
|
||
| ## 6. Clean-Slate Repo Hygiene (The "Hygiene Rule") | ||
| * **Zero-Delta Mandate**: Every new Mission (initialized via `$MISSION`) MUST start with a 0-delta `main` branch. If "Big Numbers" (large uncommitted/unmerged diffs > 100 lines) exist, the agent MUST recommend a cleanup/merge before starting new work. | ||
| * **Atomic Missions**: Every bug fix or feature MUST be its own branch and MUST be merged into `main` immediately upon verification (e.g. F5 compile in NT8). No "stacking" unrelated fixes in long-lived branches. | ||
| * **Binary & Log Purge**: Never commit `.exe`, `.log`, `.bak`, or legacy backup folders to source control. They should be stashed, deleted, or added to `.gitignore`. | ||
| * **Dashboard Cleanup**: Before ending a session, the agent MUST ensure all work is either Committed or the user has been guided to Merge. The goal is a +0/-0 dashboard between missions. | ||
|
|
||
| --- | ||
| > [!NOTE] | ||
| > This document defines **Permanent Standards**. For current active refactoring goals (e.g. Phase 6.0 Simplification), refer to the specific **Implementation Plan** or **Refactoring Roadmap** files. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The mapping now only deploys
V12_002*/V12_001files, but the script has no migration step to remove previously deployedUniversalORStrategyV12_002_Dev*andV12StandardPanel_V12_001_Devfiles, so existing NT8 environments keep stale legacy scripts side-by-side with the renamed ones. In setups that already ran older builds, this causes parallel old/new strategy code to remain compiled and can lead to running outdated logic unintentionally.Useful? React with 👍 / 👎.