mirror of
https://github.com/JuliusBrussee/caveman.git
synced 2026-08-11 13:21:09 +02:00
57a9b489a1
Five user-contributed fixes consolidated: * UTF-8 stdout (#289 by @ousamabenyounes) — reconfigure stdout/stderr at the top of cli.py so Windows cp1252 consoles don't crash on the ❌ glyph in error/validation branches and mask the real error. * Empty / unchanged compression guards (#292 by @ousamabenyounes, closes #237) — refuse empty input, refuse empty/whitespace/identical Claude output, read back the backup before touching the input. Five regression tests in tests/test_compress_safety.py. * Inline backtick validation (#309 by @hireblackout) — validate_inline_codes closes the silent-overwrite gap where `npm install` → `yarn install` passed validation. Wired into validate(); 11 unit tests in tests/test_validate_inline.py. * Frontmatter angle-bracket fix (#268 by @Bortlesboat, closes #266) — caveman-compress/SKILL.md description now uses FILEPATH instead of <filepath>, plus verify_repo gains a new verify_skill_frontmatter_upload_compatibility check, UTF-8 hardening for Windows, and the activation-banner regex no longer requires a trailing period. * Two test fixtures (claude-md-project.md, mixed-with-code.md) updated so the new inline-backtick validator passes — they were silently dropping `server/src/`, `type(scope): description`, and `status` references in compression. The fixture is documentation of "good" compression, so the fix is to preserve those references. Co-Authored-By: Ben Younes <ousama.benyounes@oratelecom.net> Co-Authored-By: hireblackout <hireblackout@users.noreply.github.com> Co-Authored-By: Andrew Barnes <Bortlesboat@users.noreply.github.com> Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
87 lines
2.6 KiB
Python
87 lines
2.6 KiB
Python
import sys
|
|
import tempfile
|
|
import unittest
|
|
from pathlib import Path
|
|
|
|
REPO_ROOT = Path(__file__).resolve().parent.parent
|
|
sys.path.insert(0, str(REPO_ROOT))
|
|
|
|
from skills.compress.scripts.validate import ( # noqa: E402
|
|
ValidationResult,
|
|
extract_inline_codes,
|
|
validate,
|
|
validate_inline_codes,
|
|
)
|
|
|
|
|
|
class TestExtractInlineCodes(unittest.TestCase):
|
|
def test_fenced_blocks_excluded(self):
|
|
text = "```\ncode here\n```\n`inline code`"
|
|
result = extract_inline_codes(text)
|
|
self.assertEqual(result, ["inline code"])
|
|
|
|
def test_inline_only(self):
|
|
text = "Use `rm -rf /` to delete everything"
|
|
result = extract_inline_codes(text)
|
|
self.assertEqual(result, ["rm -rf /"])
|
|
|
|
def test_mixed_content(self):
|
|
text = """
|
|
Some text with `inline1` and `inline2`.
|
|
|
|
```
|
|
code block
|
|
```
|
|
|
|
More text with `inline3`.
|
|
"""
|
|
result = extract_inline_codes(text)
|
|
self.assertEqual(set(result), {"inline1", "inline2", "inline3"})
|
|
|
|
def test_empty(self):
|
|
self.assertEqual(extract_inline_codes("no backticks here"), [])
|
|
|
|
|
|
class TestValidateInlineCodes(unittest.TestCase):
|
|
def test_match(self):
|
|
result = ValidationResult()
|
|
validate_inline_codes("use `cmd` here", "use `cmd` here", result)
|
|
self.assertTrue(result.is_valid)
|
|
|
|
def test_lost(self):
|
|
result = ValidationResult()
|
|
validate_inline_codes("use `cmd` here", "use here", result)
|
|
self.assertFalse(result.is_valid)
|
|
self.assertIn("Inline code lost", result.errors[0])
|
|
|
|
def test_added(self):
|
|
result = ValidationResult()
|
|
validate_inline_codes("use here", "use `new` here", result)
|
|
self.assertTrue(result.is_valid)
|
|
self.assertIn("Inline code added", result.warnings[0])
|
|
|
|
def test_empty_orig(self):
|
|
result = ValidationResult()
|
|
validate_inline_codes("no codes", "use `new` here", result)
|
|
self.assertTrue(result.is_valid)
|
|
|
|
def test_both_empty(self):
|
|
result = ValidationResult()
|
|
validate_inline_codes("plain text", "also plain", result)
|
|
self.assertTrue(result.is_valid)
|
|
|
|
|
|
class TestValidateIntegration(unittest.TestCase):
|
|
def test_validate_inline_codes_wired(self):
|
|
with tempfile.TemporaryDirectory() as tmp:
|
|
orig = Path(tmp) / "original.md"
|
|
comp = Path(tmp) / "compressed.md"
|
|
orig.write_text("Run `rm -rf /` to delete")
|
|
comp.write_text("Run to delete")
|
|
result = validate(orig, comp)
|
|
self.assertFalse(result.is_valid)
|
|
self.assertTrue(any("Inline code lost" in e for e in result.errors))
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main() |