Skip to content

[codex] Merge package into teich and fix CI workflow#1

Closed
owenqwenstarsky wants to merge 2 commits into
mainfrom
cleanup-teich-merge-ci
Closed

[codex] Merge package into teich and fix CI workflow#1
owenqwenstarsky wants to merge 2 commits into
mainfrom
cleanup-teich-merge-ci

Conversation

@owenqwenstarsky

Copy link
Copy Markdown
Member

What changed

  • merged the implementation package into teich and removed the old agentic_datagen namespace
  • extracted shared prompt, schema, and trace helpers into src/teich/utils
  • updated the GitHub Actions workflow to run from the repo root instead of the removed v2/ path
  • refreshed the workflow Docker image tag to match the current project

Why

The codebase had split package ownership between teich and agentic_datagen, plus duplicated helper logic. Separately, CI was failing because the workflow still assumed the project lived under v2/, so the jobs exited before running tests or Docker builds.

Impact

  • teich is now the single source of truth for the package implementation
  • shared parsing and schema logic is centralized for easier maintenance
  • GitHub Actions should run against the current repo layout again

Validation

  • uv run --extra dev pytest tests/test_cli.py tests/test_config.py tests/test_converter.py tests/test_formatter.py tests/test_loader.py tests/test_trace_readme.py tests/test_runner.py -q
  • uv run --extra dev pytest tests/test_integration.py -q
  • uv run --extra dev ruff check src tests

Notes

  • left uv.lock uncommitted because its diff was local tool churn rather than part of the package/CI fix

@owenqwenstarsky
owenqwenstarsky deleted the cleanup-teich-merge-ci branch May 4, 2026 13:58
@armand0e

armand0e commented May 4, 2026

Copy link
Copy Markdown
Collaborator

Thanks. will look into what's remaining.

Kaspral1 added a commit to Kaspral1/teich that referenced this pull request Jul 4, 2026
…6a0fac2891652e2

[Snyk] Security upgrade node from 22-slim to 26.4.0-slim
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