Skip to content

Chore/gh pr helper#29

Closed
mrhpython wants to merge 12 commits intomainfrom
chore/gh-pr-helper
Closed

Chore/gh pr helper#29
mrhpython wants to merge 12 commits intomainfrom
chore/gh-pr-helper

Conversation

@mrhpython
Copy link
Copy Markdown
Owner

Title:

Summary

  • What changed and why (1–3 lines).

Checklist

  • Scope is minimal and focused
  • CI green locally (if applicable)
  • Tests added/updated or not applicable
  • Docs updated or not applicable

Validation

  • Steps to verify manually:
    1.
    2.

Links

  • Related issue: #
  • Context/spec:

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 suggestions.

Reply with @codex fix comments to fix any unresolved comments.

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, or 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 fix this CI failure" or "@codex address that feedback".

Comment thread .venv/pyvenv.cfg
Comment on lines +1 to +5
home = /usr/bin
include-system-site-packages = false
version = 3.12.3
executable = /usr/bin/python3.12
command = /usr/bin/python3 -m venv /home/michael/soulfield/.venv
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[P0] Remove committed Python virtual environment

The change vendors an entire .venv tree, including machine‑specific configuration and thousands of compiled packages (for example home = /usr/bin and an absolute activation path). These files are generated artifacts that differ across platforms and dramatically inflate repository size, so every clone or PR will be noisy and potentially OS‑dependent. Please add the virtual environment to .gitignore and drop it from version control.

Useful? React with 👍 / 👎.

Comment on lines +1 to +20
# run @ 2025-09-04T18:35:22.572Z (cwd:/home/michael/soulfield/backend)

$ echo "[spec] sanity checks"
[spec] sanity checks

$ ls -alh ../workspace/research
total 40K
drwxrwxr-x 9 michael michael 4.0K Aug 24 23:03 .
drwxrwxr-x 14 michael michael 4.0K Aug 24 16:15 ..
drwxrwxr-x 2 michael michael 4.0K Aug 24 18:20 1756050649128-mobile-car-valeting-in-manchester
drwxrwxr-x 2 michael michael 4.0K Aug 24 19:28 1756060016868-online-business-ideas-uk
drwxrwxr-x 2 michael michael 4.0K Aug 24 19:35 1756060424517-online-business-ideas-uk
drwxrwxr-x 2 michael michael 4.0K Aug 24 22:52 1756072293111-integrate-https-github-com-wshobson-agents-as-agent-as-too
drwxrwxr-x 2 michael michael 4.0K Aug 24 22:57 1756072547771-infranodus-integration-for-content-gaps-https-infranodus
drwxrwxr-x 2 michael michael 4.0K Aug 24 23:01 1756072775834-bright-data-scraping-browser-integration-https-brightdata
drwxrwxr-x 2 michael michael 4.0K Aug 24 23:04 1756073037056-jina-ai-for-retrieval-rerank-over-workspace-docs-https-do
-rw-rw-r-- 1 michael michael 16 Aug 24 16:29 README.md

$ echo "[policy] show TruthLens rules"
[policy] show TruthLens rules
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] Ignore generated run logs

The commit adds run outputs under backend/.agent-os/runs/… that capture timestamps and absolute paths from a local machine. These logs are regenerated on every execution and do not participate in the application logic, so keeping them in Git will create constant diff churn and leak local environment details. Consider ignoring backend/.agent-os/runs/* and removing the checked-in logs.

Useful? React with 👍 / 👎.

@mrhpython mrhpython enabled auto-merge (squash) September 21, 2025 22:59
@mrhpython mrhpython disabled auto-merge September 21, 2025 23:06
@mrhpython mrhpython enabled auto-merge (squash) September 21, 2025 23:06
@mrhpython mrhpython disabled auto-merge September 21, 2025 23:07
@mrhpython mrhpython closed this Sep 21, 2025
@mrhpython mrhpython deleted the chore/gh-pr-helper branch September 21, 2025 23:10
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