Skip to content

Tiny fixes: escape sequence, dead code removal, docs - #190

Merged
neoneye merged 5 commits into
mainfrom
tiny-fixes
Mar 8, 2026
Merged

neoneye merged 5 commits into
mainfrom
tiny-fixes

Conversation

@neoneye

@neoneye neoneye commented Mar 8, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Fix DeprecationWarning for invalid escape sequence \> in escape_markdown.py by using a raw docstring
  • Remove unused json_repair_util.py and its test
  • Document the Literal vs Enum convention for LLM structured-output schemas in AGENTS.md
  • Add README for the presets/ directory (LM Studio presets)
  • Fix AttributeError crashes from .value / .human_readable() calls on Literal fields (3 files affected: identify_purpose.py, enrich_team_members_with_contract_type.py, data_collection.py)

Test plan

  • pytest worker_plan/worker_plan_internal/tests/test_enum_literal_parity.py -v passes with no warnings
  • python -W error -c "import worker_plan.worker_plan_internal.markdown_util.escape_markdown" confirms no escape sequence warning
  • Grep confirms json_repair_util has no remaining imports
  • Verified all Literal-migrated fields no longer use .value or Enum methods

🤖 Generated with Claude Code

neoneye and others added 5 commits March 8, 2026 02:38
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…chemas

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fields migrated from Enum to Literal are plain strings, so .value and
.human_readable() no longer exist. Fixes AttributeError at runtime.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@neoneye
neoneye merged commit 8e42dde into main Mar 8, 2026
3 checks passed
@neoneye
neoneye deleted the tiny-fixes branch March 8, 2026 02:02
@neoneye neoneye mentioned this pull request Mar 8, 2026
1 task done
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