Skip to content

Repository hygiene + STATUS diagram render fixes#14

Merged
krahd merged 3 commits into
mainfrom
chore/status-svg-rendering-sync-20260509
May 9, 2026
Merged

Repository hygiene + STATUS diagram render fixes#14
krahd merged 3 commits into
mainfrom
chore/status-svg-rendering-sync-20260509

Conversation

@krahd
Copy link
Copy Markdown
Owner

@krahd krahd commented May 9, 2026

Summary\n- remove tracked repository artifacts (sdf, config.yaml.bak)\n- fix stale config-llama augmented instruction paths\n- remove unused OLLAMA_HELPER constant\n- refresh ROADMAP 0.3.x wording\n- update modelito standalone diagram wording\n- fix STATUS diagram rendering by embedding repository SVG images\n\n## Validation\n- inline STATUS timestamp consistency verified\n- STATUS diagram references verified\n

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ab418c15b8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread run_batllm.py
"""Cross-platform launcher for BatLLM."""

from __future__ import annotations
from util.compat import require_supported_python
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Add src to sys.path before importing util.compat

When the documented python run_batllm.py command is run from a checkout without PYTHONPATH=src, this import now executes before the launcher inserts /workspace/BatLLM/src into sys.path, so Python cannot find the util package and exits with ModuleNotFoundError. The same import move appears in run_game_analyzer.py and run_tests.py, so all three top-level entry points need the sys.path setup to happen before importing util.compat.

Useful? React with 👍 / 👎.

Comment thread STATUS.md
# BatLLM Status

Last updated: 2026-05-09 23:25
Last updated: 2026-05-09 20:46
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep STATUS Last updated at the actual update time

This rewinds STATUS.md from 23:25 to 20:46 even though the commit itself was created at 2026-05-09 23:49:51 +0000; in this repository's UTC environment that makes the required Last updated marker stale immediately. AGENTS.md treats a stale or incorrectly formatted STATUS timestamp as a blocking documentation error, so this should be refreshed to the real local wall-clock update time and mirrored at the bottom of the file.

Useful? React with 👍 / 👎.

@krahd krahd merged commit b1a341f into main May 9, 2026
5 checks passed
@krahd krahd deleted the chore/status-svg-rendering-sync-20260509 branch May 9, 2026 23:52
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.

1 participant