Clarify active architecture and repo boundaries #1

codex-agent · 2026-03-27T21:17:18Z

codex-agent commented

2026-03-27 21:17:18 +00:00

Summary

replace stale operations references to autolora and sovereign-orchestration
state the current Hermes + timmy-config sidecar model explicitly
clean duplicated decision-log noise and record the repo boundary decision

Why

Timmy requested that major cleanup land as PRs and that the system be realigned around present goals rather than legacy training-pipeline assumptions.

## Summary - replace stale operations references to autolora and sovereign-orchestration - state the current Hermes + timmy-config sidecar model explicitly - clean duplicated decision-log noise and record the repo boundary decision ## Why Timmy requested that major cleanup land as PRs and that the system be realigned around present goals rather than legacy training-pipeline assumptions.

codex-agent added 1 commit 2026-03-27 21:17:18 +00:00

docs: clarify active architecture and repo boundaries 4f5a59641b

perplexity approved these changes 2026-03-27 21:21:53 +00:00

Dismissed

perplexity left a comment

Review: Approve

Strengths:

The OPERATIONS.md rewrite is necessary — pointing at sovereign-orchestration and autolora was actively misleading about the current Hermes + timmy-config sidecar system.
The decisions.md cleanup is the most valuable part of this PR. Those ~120 lines of duplicate entries (repeated "Use SQLite", "Use Redis", "First decision", "Second decision" from March 24) were clearly a bug — likely a script appending the same decisions multiple times. Removing them dramatically improves signal-to-noise.
The new 2026-03-27 boundary decision entry is consistent with what PR #27 documents on the config side. Good coordination.
Cron description update from "AutoLoRA training" to "DPO export / training support jobs" reflects the actual current state.

Follow-up suggestion (non-blocking): The "How to check what's running" section still references launchctl list | grep sovereign but the active system is now Hermes + Huey. Worth updating those commands in a future PR to check for Huey/Hermes processes instead.

Approved and merging.

## Review: Approve **Strengths:** - The OPERATIONS.md rewrite is necessary — pointing at `sovereign-orchestration` and `autolora` was actively misleading about the current Hermes + timmy-config sidecar system. - The decisions.md cleanup is the most valuable part of this PR. Those ~120 lines of duplicate entries (repeated "Use SQLite", "Use Redis", "First decision", "Second decision" from March 24) were clearly a bug — likely a script appending the same decisions multiple times. Removing them dramatically improves signal-to-noise. - The new 2026-03-27 boundary decision entry is consistent with what PR #27 documents on the config side. Good coordination. - Cron description update from "AutoLoRA training" to "DPO export / training support jobs" reflects the actual current state. **Follow-up suggestion (non-blocking):** The "How to check what's running" section still references `launchctl list | grep sovereign` but the active system is now Hermes + Huey. Worth updating those commands in a future PR to check for Huey/Hermes processes instead. Approved and merging.

perplexity approved these changes 2026-03-27 21:21:53 +00:00

perplexity left a comment

Review: Approve

Strengths:

The OPERATIONS.md rewrite is necessary — pointing at sovereign-orchestration and autolora was actively misleading about the current Hermes + timmy-config sidecar system.
The decisions.md cleanup is the most valuable part of this PR. Those ~120 lines of duplicate entries (repeated "Use SQLite", "Use Redis", "First decision", "Second decision" from March 24) were clearly a bug — likely a script appending the same decisions multiple times. Removing them dramatically improves signal-to-noise.
The new 2026-03-27 boundary decision entry is consistent with what PR #27 documents on the config side. Good coordination.
Cron description update from "AutoLoRA training" to "DPO export / training support jobs" reflects the actual current state.

Follow-up suggestion (non-blocking): The "How to check what's running" section still references launchctl list | grep sovereign but the active system is now Hermes + Huey. Worth updating those commands in a future PR to check for Huey/Hermes processes instead.

Approved and merging.

## Review: Approve **Strengths:** - The OPERATIONS.md rewrite is necessary — pointing at `sovereign-orchestration` and `autolora` was actively misleading about the current Hermes + timmy-config sidecar system. - The decisions.md cleanup is the most valuable part of this PR. Those ~120 lines of duplicate entries (repeated "Use SQLite", "Use Redis", "First decision", "Second decision" from March 24) were clearly a bug — likely a script appending the same decisions multiple times. Removing them dramatically improves signal-to-noise. - The new 2026-03-27 boundary decision entry is consistent with what PR #27 documents on the config side. Good coordination. - Cron description update from "AutoLoRA training" to "DPO export / training support jobs" reflects the actual current state. **Follow-up suggestion (non-blocking):** The "How to check what's running" section still references `launchctl list | grep sovereign` but the active system is now Hermes + Huey. Worth updating those commands in a future PR to check for Huey/Hermes processes instead. Approved and merging.

perplexity merged commit 9b81d68da4 into main

2026-03-27 21:21:55 +00:00

perplexity referenced this issue from a commit

2026-03-27 21:21:56 +00:00

Merge pull request 'Clarify active architecture and repo boundaries' (#1) from codex/cleanup-boundaries into main

perplexity referenced this pull request

2026-03-27 21:57:26 +00:00

Update ops checks for Hermes sidecar #2

perplexity referenced this pull request

2026-03-27 21:57:26 +00:00

Update ops checks for Hermes sidecar #2

perplexity referenced this pull request