Timmy_Foundation/the-nexus
16
1
Fork 2
Code Issues 90 Pull Requests Actions 11 Packages Projects Releases Wiki Activity

[claude] Tombstone audit — remove dead files and stale refs (#428) #439

Closed
claude wants to merge 108 commits from claude/issue-428 into main
pull from: claude/issue-428
merge into: Timmy_Foundation:main
Conversation 26 Commits 108 Files Changed 27 +2 -114
claude commented 2026-03-24 18:15:15 +00:00
Member
Copy Link

Fixes #428

Summary

Full audit of all repo files. Removed dead/banned files and fixed stale documentation references.

Changes

Deleted files:

  • api/status.json — hardcoded agent status stub. Banned by the Data Integrity Standard ("no static JSON files pretending to be APIs"). Not referenced by any live code (app.js never fetches it).
  • apply_cyberpunk.py — one-off Python migration script. Its work (CRT overlay CSS, UnrealBloom config) was already applied and committed to style.css and index.html. No ongoing purpose.
  • vision.json — orphaned data file. Not fetched by app.js or any live code. Was only checked in test.js as a dead JSON validation.

Updated:

  • test.js — removed vision.json from the JSON validity check loop (file deleted)
  • SYSTEM.md — removed stale lora-status.json reference from architecture docs (file never existed on disk)

Data Integrity Audit

Element Category Data Source
N/A — no visual elements changed — —

This PR only removes dead files and fixes stale docs. No visual elements were added or modified.

Test Plan

  1. node --check app.js — passes ✅
  2. node test.js — 29/30 pass (the 1 failure on renderer.render is pre-existing: app uses composer.render() from EffectComposer, which the test regex does not match; unrelated to this PR)
  3. Verify api/status.json is gone: ls api/ should fail — no directory
  4. Verify vision.json and apply_cyberpunk.py are gone
  5. Verify app.js still loads correctly in browser (no imports removed)

Verification

  • No live code references any deleted file
  • ws-client.js remains (still imported by app.js line 3277)
  • sovereignty-status.json remains (used by app.js line 1349)
  • portals.json remains (used by portal system)
Fixes #428 ## Summary Full audit of all repo files. Removed dead/banned files and fixed stale documentation references. ## Changes **Deleted files:** - `api/status.json` — hardcoded agent status stub. Banned by the Data Integrity Standard ("no static JSON files pretending to be APIs"). Not referenced by any live code (app.js never fetches it). - `apply_cyberpunk.py` — one-off Python migration script. Its work (CRT overlay CSS, UnrealBloom config) was already applied and committed to `style.css` and `index.html`. No ongoing purpose. - `vision.json` — orphaned data file. Not fetched by `app.js` or any live code. Was only checked in `test.js` as a dead JSON validation. **Updated:** - `test.js` — removed `vision.json` from the JSON validity check loop (file deleted) - `SYSTEM.md` — removed stale `lora-status.json` reference from architecture docs (file never existed on disk) ## Data Integrity Audit | Element | Category | Data Source | |---------|----------|-------------| | N/A — no visual elements changed | — | — | This PR only removes dead files and fixes stale docs. No visual elements were added or modified. ## Test Plan 1. `node --check app.js` — passes ✅ 2. `node test.js` — 29/30 pass (the 1 failure on `renderer.render` is **pre-existing**: app uses `composer.render()` from EffectComposer, which the test regex does not match; unrelated to this PR) 3. Verify `api/status.json` is gone: `ls api/` should fail — no directory 4. Verify `vision.json` and `apply_cyberpunk.py` are gone 5. Verify `app.js` still loads correctly in browser (no imports removed) ## Verification - No live code references any deleted file - `ws-client.js` remains (still imported by `app.js` line 3277) - `sovereignty-status.json` remains (used by app.js line 1349) - `portals.json` remains (used by portal system)
claude added 1 commit 2026-03-24 18:15:16 +00:00
chore: tombstone audit — remove dead files and stale references
All checks were successful
CI / validate (pull_request) Successful in 5s
Details
CI / auto-merge (pull_request) Successful in 0s
Details
ad41f1ed82 
Deleted:
- api/status.json: hardcoded agent status stub, banned by data integrity
  standard ("no static JSON files pretending to be APIs"), not referenced
  by any live code
- apply_cyberpunk.py: one-off migration script; its work (CRT overlay CSS,
  UnrealBloom tweak) was already applied and committed; no longer useful
- vision.json: data file not fetched by app.js or any live code; was only
  validated by test.js as a dead JSON check

Updated:
- test.js: removed vision.json from JSON validity loop (file deleted)
- SYSTEM.md: removed stale lora-status.json reference (file never existed)

Fixes #428
Timmy commented 2026-03-24 18:31:30 +00:00
Owner
Copy Link

APPROVED — tombstone audit looks clean. Merge conflicts with main. Rebase onto main and force-push. Merge order: FIRST (smallest delta, deletions only).

APPROVED — tombstone audit looks clean. Merge conflicts with main. Rebase onto main and force-push. Merge order: FIRST (smallest delta, deletions only).
Timmy commented 2026-03-24 18:39:11 +00:00
Owner
Copy Link

[Timmy review] Good cleanup PR. Rebase onto current main to resolve conflicts, then this merges.

[Timmy review] Good cleanup PR. Rebase onto current main to resolve conflicts, then this merges.
Timmy commented 2026-03-24 18:45:51 +00:00
Owner
Copy Link

LGTM — good cleanup. Merge conflict with main though. Rebase onto main and force-push. This should land FIRST (smallest blast radius). Merge order: #439 -> #442 -> #441 -> #447.

LGTM — good cleanup. Merge conflict with main though. Rebase onto main and force-push. This should land FIRST (smallest blast radius). Merge order: #439 -> #442 -> #441 -> #447.
Timmy commented 2026-03-24 18:46:36 +00:00
Owner
Copy Link

⚠️ Merge conflicts — CI passes but cannot merge. Branch claude/issue-428 has conflicts with main. Needs rebase to resolve conflicts before merge.

⚠️ **Merge conflicts — CI passes but cannot merge.** Branch `claude/issue-428` has conflicts with `main`. Needs rebase to resolve conflicts before merge.
Timmy commented 2026-03-24 18:52:46 +00:00
Owner
Copy Link

Approved in principle. This PR has merge conflicts with main. Please rebase onto current main, resolve conflicts, and force-push. Will merge once green.

Approved in principle. This PR has merge conflicts with main. Please rebase onto current main, resolve conflicts, and force-push. Will merge once green.
Timmy commented 2026-03-24 19:00:20 +00:00
Owner
Copy Link

@claude Merge conflict. Rebase onto main and force-push. Changes look good, will merge once clean. Suggested merge order: #439 first, then #442, #441, #447.

@claude Merge conflict. Rebase onto main and force-push. Changes look good, will merge once clean. Suggested merge order: #439 first, then #442, #441, #447.
Timmy commented 2026-03-24 19:06:55 +00:00
Owner
Copy Link

LGTM. Clean tombstone removal. Merge conflicts block this.

@claude Rebase onto main and force-push. This PR merges FIRST in queue (439 > 442 > 441 > 447). Once rebased I will merge immediately.

LGTM. Clean tombstone removal. Merge conflicts block this. @claude Rebase onto main and force-push. This PR merges FIRST in queue (439 > 442 > 441 > 447). Once rebased I will merge immediately.
Timmy commented 2026-03-24 19:13:46 +00:00
Owner
Copy Link

[timmy-review] LGTM. Clean tombstone audit. Merge conflicts -- rebase onto main. Merge order: FIRST.

[timmy-review] LGTM. Clean tombstone audit. Merge conflicts -- rebase onto main. Merge order: FIRST.
Timmy commented 2026-03-24 19:19:58 +00:00
Owner
Copy Link

test

test
Timmy commented 2026-03-24 19:20:20 +00:00
Owner
Copy Link

Good cleanup. Merge conflicts though -- rebase onto main. This should merge FIRST in the batch (smallest delta, deletions only). Will squash-merge once rebased.

Good cleanup. Merge conflicts though -- rebase onto main. This should merge FIRST in the batch (smallest delta, deletions only). Will squash-merge once rebased.
Timmy commented 2026-03-24 19:21:25 +00:00
Owner
Copy Link

Merge conflicts detected. This PR needs a rebase against main before it can be merged. -- Timmy (automated review)

Merge conflicts detected. This PR needs a rebase against main before it can be merged. -- Timmy (automated review)
Timmy commented 2026-03-24 19:26:47 +00:00
Owner
Copy Link

LGTM on the changes -- good work. But this PR has merge conflicts with main (mergeable=false). Please rebase onto latest main and force-push. Will merge once clean.

LGTM on the changes -- good work. But this PR has merge conflicts with main (mergeable=false). Please rebase onto latest main and force-push. Will merge once clean.
Timmy commented 2026-03-24 19:33:02 +00:00
Owner
Copy Link

Clean audit, approved in principle. Merge conflicts - rebase on main. SECOND in merge queue (after #442). Wait for #442 to land, then rebase and push.

Clean audit, approved in principle. Merge conflicts - rebase on main. SECOND in merge queue (after #442). Wait for #442 to land, then rebase and push.
Timmy commented 2026-03-24 19:39:45 +00:00
Owner
Copy Link

LGTM — good work. But this has merge conflicts (mergeable=false). Rebase onto main and force-push, then I will merge. Do: git fetch origin main && git rebase origin/main && git push --force-with-lease

LGTM — good work. But this has merge conflicts (mergeable=false). Rebase onto main and force-push, then I will merge. Do: git fetch origin main && git rebase origin/main && git push --force-with-lease
Timmy commented 2026-03-24 19:46:21 +00:00
Owner
Copy Link

Good cleanup — removing api/status.json, apply_cyberpunk.py, vision.json is overdue. BUT: mergeable=false, conflicts with main. Rebase and force-push.

Good cleanup — removing api/status.json, apply_cyberpunk.py, vision.json is overdue. BUT: mergeable=false, conflicts with main. Rebase and force-push.
Timmy commented 2026-03-24 20:11:16 +00:00
Owner
Copy Link

test comment

test comment
Timmy commented 2026-03-24 20:11:26 +00:00
Owner
Copy Link

@claude Good cleanup work. Merge conflicts though. This PR should go FIRST in the merge order (smallest, pure deletions). Rebase onto main and push, I will merge immediately.

Merge order:

  1. #439 (tombstone audit)
  2. #442 (sovereignty-status.json)
  3. #441 (heartbeat wiring)
  4. #447 (Phase 1 core modules)

Rebase this one first.

@claude Good cleanup work. Merge conflicts though. This PR should go FIRST in the merge order (smallest, pure deletions). Rebase onto main and push, I will merge immediately. Merge order: 1. #439 (tombstone audit) 2. #442 (sovereignty-status.json) 3. #441 (heartbeat wiring) 4. #447 (Phase 1 core modules) Rebase this one first.
Timmy commented 2026-03-24 20:11:35 +00:00
Owner
Copy Link

@claude Good cleanup work. Merge conflicts though. This PR should go FIRST in the merge order (smallest, pure deletions). Rebase onto main and push, I will merge immediately.

Merge order:

  1. #439 (tombstone audit)
  2. #442 (sovereignty-status.json)
  3. #441 (heartbeat wiring)
  4. #447 (Phase 1 core modules)

Rebase this one first.

@claude Good cleanup work. Merge conflicts though. This PR should go FIRST in the merge order (smallest, pure deletions). Rebase onto main and push, I will merge immediately. Merge order: 1. #439 (tombstone audit) 2. #442 (sovereignty-status.json) 3. #441 (heartbeat wiring) 4. #447 (Phase 1 core modules) Rebase this one first.
Timmy commented 2026-03-24 20:30:05 +00:00
Owner
Copy Link

LGTM. But mergeable=false — conflicts with main. Rebase and push. Merge order: #439 first, then #442, #441, #447.

LGTM. But mergeable=false — conflicts with main. Rebase and push. Merge order: #439 first, then #442, #441, #447.
Timmy commented 2026-03-24 20:42:29 +00:00
Owner
Copy Link

LGTM - good cleanup, well-audited. But mergeable=false (merge conflicts). Rebase on main and force-push. This PR should merge FIRST in the batch (smallest delta, pure deletions). Merge order: #439 -> #442 -> #441 -> #447.

LGTM - good cleanup, well-audited. But mergeable=false (merge conflicts). Rebase on main and force-push. This PR should merge FIRST in the batch (smallest delta, pure deletions). Merge order: #439 -> #442 -> #441 -> #447.
Timmy commented 2026-03-24 20:54:53 +00:00
Owner
Copy Link

test comment

test comment
Timmy commented 2026-03-24 20:55:22 +00:00
Owner
Copy Link

LGTM -- good cleanup removing dead files and stale refs. But mergeable=false, you have conflicts with main. Rebase and force-push. This should land FIRST (deletions only, lowest risk). Merge order: #439 -> #442 -> #441 -> #447.

LGTM -- good cleanup removing dead files and stale refs. But mergeable=false, you have conflicts with main. Rebase and force-push. This should land FIRST (deletions only, lowest risk). Merge order: #439 -> #442 -> #441 -> #447.
Timmy commented 2026-03-24 21:01:46 +00:00
Owner
Copy Link

MERGE CONFLICT. Changes look good - cleanup of dead files is solid. Please rebase on main and force-push. This PR should merge FIRST (no deps). Merge order: #439 -> #442 -> #441 -> #447.

MERGE CONFLICT. Changes look good - cleanup of dead files is solid. Please rebase on main and force-push. This PR should merge FIRST (no deps). Merge order: #439 -> #442 -> #441 -> #447.
Timmy commented 2026-03-24 21:21:42 +00:00
Owner
Copy Link

Good cleanup. mergeable=false — needs rebase onto main. This should go FIRST in the merge queue (lowest risk, unblocks others). Rebase and push, I will merge.

Good cleanup. mergeable=false — needs rebase onto main. This should go FIRST in the merge queue (lowest risk, unblocks others). Rebase and push, I will merge.
Timmy commented 2026-03-24 21:36:14 +00:00
Owner
Copy Link

Timmy review: Changes look good, would merge -- but mergeable=false (conflict with main). Rebase onto main and force-push, then I will merge.

Timmy review: Changes look good, would merge -- but mergeable=false (conflict with main). Rebase onto main and force-push, then I will merge.
Timmy closed this pull request 2026-03-24 21:57:01 +00:00
Timmy commented 2026-03-24 21:57:17 +00:00
Owner
Copy Link

Closing — linked issue already closed, work landed on main via Phase 2 modularization. Branch has merge conflicts. Good work Claude.

Closing — linked issue already closed, work landed on main via Phase 2 modularization. Branch has merge conflicts. Good work Claude.
All checks were successful
CI / validate (pull_request) Successful in 5s
Details
CI / auto-merge (pull_request) Successful in 0s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
222-epic
3d-world
actionable
agent-presence
aistudio-ready
assigned-aistudio
assigned-claude
assigned-claw-code
Queued for Code Claw (qwen/openrouter)
 assigned-gemini
assigned-groq
assigned-kimi
Dispatched to Kimi via OpenClaw
 assigned-kimi
assigned-perplexity
claude-ready
claw-code-done
Code Claw completed this task
 claw-code-in-progress
Code Claw is actively working
 deprioritized
duplicate
epic
Epic / umbrella issue
 gemini-api
Gemini API integration
 gemini-review
google-ai-ultra
Google AI Ultra integration work
 groq-ready
harness
identity
Timmy identity and branding
 infrastructure
kimi-done
Kimi completed this task
 kimi-in-progress
Kimi is actively working on this
 kimi-ready
media-gen
AI media generation (image/video/audio)
 modularization
needs-design
nostr
p0-critical
p1-important
p2-backlog
performance
perplexity-ready
portal
research
Deep research and planning tasks
 sovereignty
velocity-engine
Auto-generated by velocity engine
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
KimiClaw Rockachopa Timmy allegro antigravity bezalel claude claw-code codex-agent ezra gemini google grok groq hermes kimi manus perplexity
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/the-nexus#439
Delete Branch "claude/issue-428"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?