Skip to content

fix(process): handle CRLF line endings and improve subcategory classification#10

Merged
epicsagas merged 1 commit into
mainfrom
worktree-orbit-fix-process-body-wipe-and-subcategory
May 16, 2026
Merged

fix(process): handle CRLF line endings and improve subcategory classification#10
epicsagas merged 1 commit into
mainfrom
worktree-orbit-fix-process-body-wipe-and-subcategory

Conversation

@epicsagas

Copy link
Copy Markdown
Owner

Summary

Fix of process-all body wipe bug (DEBT-006) and improve PARA subcategory classification (DEBT-007).

Spec

  • ID: SPEC-1778653805
  • Pipeline: PIPELINE-1778653805
  • Requirements: R1, R2, R3, R4

Changes

  • split_frontmatter: normalize CRLF to LF before regex matching
  • split_frontmatter: make trailing \n after closing --- optional
  • default_category prompt: add explicit subcategory/detail selection rules per type
  • Add 8 regression tests (4 for DEBT-006, 4 for DEBT-007)

Acceptance Criteria Verified

  • AC1: ✅ CRLF files retain full body content
  • AC2: ✅ Files without trailing newline retain full body content
  • AC3: ✅ AI-returned subcategory/detail used in resolve_confirmed_targets
  • AC4: ✅ ConceptSeed uses AI-returned subcategory
  • AC5: ✅ 74/74 tests passing
  • AC6: ✅ clippy + fmt clean

Check Report

Dimension Result
Code Quality PASS
Security PASS
Performance PASS
Tests 74/74

Test Plan

  • Unit tests pass
  • clippy -- -D warnings clean
  • cargo fmt --check clean
  • Manual: run of process-all on a CRLF inbox file and verify body preserved
  • Manual: verify AI returns meaningful subcategory for Resource/ConceptSeed notes

…fication

- normalize CRLF to LF before frontmatter regex matching (DEBT-006)
- make trailing newline after closing --- optional
- add explicit subcategory/detail selection rules to AI prompt (DEBT-007)
- add 8 regression tests for both fixes
@epicsagas epicsagas merged commit 5f5cbcb into main May 16, 2026
10 checks passed
@epicsagas epicsagas deleted the worktree-orbit-fix-process-body-wipe-and-subcategory branch May 16, 2026 10:08
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