Skip to content

Commit 55fe74e

Browse files
Merge pull request #18 from CoreyLeath-code/codex/root-directory-cleanup
chore: clean up ambiguous root files
2 parents 0a570b2 + 1731b3e commit 55fe74e

5 files changed

Lines changed: 32 additions & 8 deletions

File tree

‎.dockerignore‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ LICENSE
5050
DailyLog.md
5151
Contributing.md
5252
system_design.md
53-
FILE Structure
5453
Metrics.md
5554

5655
# Terraform state (contains sensitive data)

‎Test‎

Lines changed: 0 additions & 6 deletions
This file was deleted.

FILE Structure renamed to docs/reference/historical-repository-layout.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
# Historical repository layout reference
2+
3+
This manually maintained snapshot was moved from the repository root during the 2026-08-05 root-directory audit. It is retained for historical context only and is **not** a generated current inventory; use the repository tree and service documentation for current paths.
4+
5+
## Original snapshot
6+
7+
```text
18
SentinelAI/
29
│
310
├── ingestion-service/ # Go - Log ingestion + API gateway
@@ -44,4 +51,5 @@ SentinelAI/
4451
│ └── streamlit_app.py
4552
├── src/
4653
├── requirements.txt
47-
├── README.md
54+
├── README.md
55+
```
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# Root directory audit
2+
3+
Reviewed on 2026-08-05. This record captures the reference searches and disposition of ambiguous root files.
4+
5+
## Moved
6+
7+
| Original file | Canonical location | Reason |
8+
| --- | --- | --- |
9+
| `FILE Structure` | `docs/reference/historical-repository-layout.md` | Unique, manually maintained historical layout reference; moved from a space-containing root filename. |
10+
| `Prometheus ServiceMonitor (Metrics Credibility)` | `k8s/sentinelai-backend-servicemonitor.yaml` | Unique `ServiceMonitor` manifest; Kubernetes manifests belong under `k8s/`. |
11+
12+
## Removed duplicate
13+
14+
| Removed file | Canonical source | Verification |
15+
| --- | --- | --- |
16+
| `Test` | `tests/` directory | The root file was only a five-file snapshot of the existing test tree. Repository searches found no build, CI, Docker, script, documentation, or source reference to the root snapshot. |
17+
18+
## NEEDS HUMAN DECISION
19+
20+
| Filename | Why deletion is uncertain | Recommended action |
21+
| --- | --- | --- |
22+
| `bindings.cpp` | It is a unique pybind11 module declaration for `ingestion_cpp`; no `ingestion_cpp/` implementation or build reference was found, so its intended integration cannot be proven. | Identify or add the owning extension build and implementation; then either move it with that component or remove it in a dedicated, build-validated PR. |
23+
| `drift_engine.cpp` | It is not identical to `drift-engine/drift_engine.cpp`: the root program is a PSI-only sample, while the service implementation parses JSON and computes PSI/KS. `docker/Dockerfile` explicitly compiles the root file. | Choose one supported drift-engine image/build path, update the Docker/build/documentation references, then remove the other implementation only after an image build and behavior test. |
File renamed without changes.

0 commit comments

Comments
 (0)