Skip to content

PR #78 Part 1: Add D_MINECRAFT_SPATIAL + D_AI_ONTOLOGICAL_STATUS; redact hardcoded API key#78

Merged
aidoruao merged 4 commits into
mainfrom
copilot/part-1-pr-78-spec
Mar 29, 2026
Merged

PR #78 Part 1: Add D_MINECRAFT_SPATIAL + D_AI_ONTOLOGICAL_STATUS; redact hardcoded API key#78
aidoruao merged 4 commits into
mainfrom
copilot/part-1-pr-78-spec

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 29, 2026

  • Audit existing codebase — F_MINECRAFT_004 confirmed missing
  • Add tests/test_f_minecraft_004.py — 4 tests verifying SigmaLoraConstraintSystem rejects spatially-violating commands (GRACE rejects "dig straight down", KENOSIS rejects "mine forever", valid command passes, allow_autonomous override works)
  • Add F_MINECRAFT_004 entry in ontology/falsification_tests.json (total_tests 102→103)
  • Wire F_MINECRAFT_004 into ontology/ontology.json (OI_MINECRAFT_001) and ontology/case_studies.json (CS_MINECRAFT_001 falsification_tests, all-F-IDs list, methodology_components)
  • Add TurtlePositionTracker + sign reading to minimal_ai_ide/brain_bridge_constrained.lua ({x,y,z,facing} state, turtle.inspect() sign parsing, position/sign context merged into every constraint-server request, status command shows dead-reckoning position)
  • Create transcripts/ directory with ChatGPT, DeepSeek, Gemini session logs (CS_AIONT_001 evidence — makes case study verifiable, includes Gemini key quote)
  • All 9 tests pass (F_MINECRAFT_001–004, F_AIONT_001–002)

⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

@aidoruao
Copy link
Copy Markdown
Owner

@copilot "Found it. The bug is on line 1885 of ontology/ontology.json.

The Part 3 commit removed resolution_notes from OI_MINECRAFT_001 but left the trailing comma on the line above. The file currently reads:

      "falsification_tests": [
        "F_MINECRAFT_001",
        "F_MINECRAFT_002",
        "F_MINECRAFT_003",
        "F_MINECRAFT_004"
      ],       ← trailing comma — falsification_tests is now the LAST property
    },
``` [10-cite-0](#10-cite-0) 

`falsification_tests` used to be followed by `resolution_notes`, so the comma was valid. When the commit deleted `resolution_notes`, the comma became a trailing comma before `}`, which is illegal JSON.

**Fix:** line 1885, change `],` to `]`:

```diff
-      ],
+      ]
     },

That's it — one character. The CI JSON validation step (python -m json.tool ontology/ontology.json) will pass after this." fix these issues thank you

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 29, 2026

@copilot "Found it. The bug is on line 1885 of ontology/ontology.json.

The Part 3 commit removed resolution_notes from OI_MINECRAFT_001 but left the trailing comma on th...

Fixed in ed765c4 — removed the trailing comma from line 1885, python3 -m json.tool ontology/ontology.json now validates cleanly.

@aidoruao aidoruao marked this pull request as ready for review March 29, 2026 18:56
@aidoruao aidoruao merged commit e4408fa into main Mar 29, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants