darthrubik
fbd70ff8f9
Adding another test
2026-05-20 16:22:05 -05:00
darthrubik
adfb070e6c
Skill Tool Implementation & XML Output
...
- Added make_skill_tool() which looks up requested skills in an enabled map.
- Tool returns XML-wrapped string with <skill_content> and <skill_files>.
- Skill files are returned as absolute paths.
- Added SkillToolParams to json_types.hpp.
- Registered skill_tool in main.cpp with the enabled map built from SkillScanner.
- Added test_tool_skill.sh to verify XML structure and absolute paths.
- Updated CMakeLists.txt and tests/CMakeLists.txt accordingly.
2026-05-20 16:22:02 -05:00
darthrubik
2c43fd3c5d
CLI Flag -S (List Skills)
...
- Added -S flag to print the list of discovered skills (name + description)
- Modified SkillScanner to store parsed SkillMetadata alongside paths
- Added print_skills() method to SkillScanner
- Added fallback to filename stem when frontmatter parsing fails
- Updated help text to document the -S flag
- Added test_skill_listing.sh to verify the -S flag functionality
Acceptance criteria met:
- -S prints the list of valid skills
- Output is clean and readable (name - description format)
2026-05-20 16:22:00 -05:00
darthrubik
c9461f2e88
YAML Frontmatter Parsing & Debug Parsing
...
- Add skill_parsing.hpp and skill_parsing.cpp for YAML frontmatter parsing
- Extract name (required) and description (optional) from SKILL.md files
- Add -y flag to print parsed metadata and exit
- Add test_skill_parsing.sh to verify the -y flag functionality
- Update CMakeLists.txt to include the new source and test
2026-05-20 16:21:58 -05:00
darthrubik
244ecc1d32
Tweaking tests a little
2026-05-20 16:21:58 -05:00
darthrubik
6c898ca415
Implement directory scanning and debug listing for skills
...
- Add SkillScanner class to recursively find SKILL.md files
- Scan standard global (~/.local/share/harness/skills/, ~/.config/harness/skills/) and local (./skills/, ./.skills/) paths
- Add -D flag to print all discovered file paths and exit
- Add unit test for scanner functionality
- Update CMakeLists.txt and main.cpp accordingly
2026-05-20 16:21:49 -05:00
darthrubik
9f55cc10a2
Merge pull request 'Prompt output should be stderr' ( #23 ) from bugfix/prompt-output-should-bestderr into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/23
2026-05-20 14:39:38 -05:00
darthrubik
f8aaaa22d5
Prompt output should be stderr
2026-05-19 16:22:27 -05:00
darthrubik
89db52cbe4
Merge pull request 'Adding contributing guidelines' ( #22 ) from feature/contributing into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/22
2026-05-19 16:21:31 -05:00
darthrubik
07f12b359f
Adding contributing guidelines
2026-05-19 16:18:59 -05:00
darthrubik
82886349bd
Merge pull request 'Printout prompt that was input for better user experience' ( #21 ) from feature/printout-input-prompt into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/21
2026-05-19 15:13:59 -05:00
darthrubik
286815d1d3
Printout prompt that was input for better user experience
2026-05-19 15:12:14 -05:00
darthrubik
6dffe9a105
Merge pull request 'Adding editor support when not using -m' ( #20 ) from feature/editor into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/20
2026-05-13 15:41:32 -05:00
darthrubik
8f2cc0aabd
Merge remote-tracking branch 'origin/master' into feature/editor
2026-05-13 15:41:15 -05:00
darthrubik
ca2b0effd4
Updating README
2026-05-13 15:34:14 -05:00
darthrubik
c5f4fd2cf0
Adding editor support when not using -m
2026-05-13 15:26:03 -05:00
darthrubik
466b738243
Merge pull request 'Unhardcode build directory' ( #19 ) from feature/unhardcode-build-dir into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/19
2026-05-13 14:39:41 -05:00
darthrubik
dbe84aea09
Unhardcode build directory
2026-05-13 14:36:37 -05:00
darthrubik
cdf2868063
Merge pull request 'Adding usage data printout' ( #18 ) from feature/usage-data-printout into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/18
2026-05-12 14:23:07 -05:00
darthrubik
de7f4d17ba
Merge pull request 'Safe shell' ( #17 ) from feature/safe-shell into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/17
2026-05-12 14:19:53 -05:00
darthrubik
5d6a102b30
Safe shell
2026-05-12 14:15:55 -05:00
darthrubik
9fec6198cf
Adding usage data printout
2026-05-12 06:25:40 -05:00
darthrubik
e7df4cd50f
Merge pull request 'Refactoring cmake test logic to be easier to make a test' ( #16 ) from feature/refactor-cmake-test into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/16
2026-05-11 15:59:19 -05:00
darthrubik
9cef36fdde
Refactoring cmake test logic to be easier to make a test
2026-05-11 15:57:49 -05:00
darthrubik
bd94fb9cfc
Merge pull request 'Handling llama-cpp 'reasoning_content' vs ollama 'reasoning'' ( #15 ) from bugfix/reasoning-content-unhandled into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/15
2026-05-11 15:52:41 -05:00
darthrubik
aa6bfdd01e
Handling llama-cpp 'reasoning_content' vs ollama 'reasoning'
...
Ollama sends reasoning in "reasoning" field while llama-cpp in
"reasoning_content" field. This means that when sending back up to the
server we weren't using reasoning feedback
2026-05-11 15:49:35 -05:00
darthrubik
0c0dedeefb
Merge pull request 'feature/typesafe_json' ( #14 ) from feature/typesafe_json into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/14
2026-05-11 15:21:06 -05:00
darthrubik
3d31072123
Fixing broken refactor json tools to std::vector<string> instead of std::vector<Tool>
2026-05-11 15:13:35 -05:00
darthrubik
b60893d4c0
Silence warning about missing initializers
2026-05-11 15:01:55 -05:00
darthrubik
0761987038
Using NLOHMANN_DEFINE_TYPE_INTRUSIVE_WITH_DEFAULT instead of custom parsers
2026-05-11 15:01:55 -05:00
darthrubik
6ede248f50
Refactoring a lot more json stuff
2026-05-11 15:01:55 -05:00
darthrubik
efc0af42f7
First pass json cleanup
2026-05-11 15:01:51 -05:00
darthrubik
978f9c8e7d
Merge pull request 'feature/test-timeout' ( #13 ) from feature/test-timeout into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/13
2026-05-11 09:58:05 -05:00
darthrubik
7e2921b324
Testing timeout ability of run_shell
2026-05-11 09:33:58 -05:00
darthrubik
586cd55ab4
Removing outdated comment about buggy code
2026-05-11 09:32:04 -05:00
darthrubik
7e4051b822
Merge pull request 'Fixing non-determinism bug in shell tool' ( #12 ) from bugfix/non-deterministic-shell into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/12
2026-05-09 12:51:54 -05:00
darthrubik
757835758d
Fixing non-determinism bug in shell tool
2026-05-09 12:44:50 -05:00
darthrubik
3eeba859b8
Merge pull request 'feature/more-tests' ( #11 ) from feature/more-tests into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/11
2026-05-09 12:35:50 -05:00
darthrubik
a5f740448f
Merge remote-tracking branch 'origin/master' into feature/more-tests
2026-05-09 12:35:33 -05:00
darthrubik
2889a21eb6
Refactoring tests a little
2026-05-09 12:26:08 -05:00
darthrubik
19983b2aa8
Merge pull request 'Fixing bug with subagents cleanup curl state before it was done being used' ( #10 ) from bugfix/curl-state into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/10
2026-05-09 11:35:05 -05:00
darthrubik
be79d5a040
Merge pull request 'Fail on tool not existing' ( #9 ) from feature/fail-on-no-tool into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/9
2026-05-09 11:26:50 -05:00
darthrubik
999d2bbcc0
Fixing bug with subagents cleanup curl state before it was done being used
2026-05-09 11:17:08 -05:00
darthrubik
1adc8deb8b
Fail on tool not existing
2026-05-09 10:58:20 -05:00
darthrubik
4cbdcabae8
First pass new tests
2026-05-09 10:31:10 -05:00
darthrubik
757a9d3329
Removing colorized output :( so that it supports non-gnu diff
2026-05-09 10:30:51 -05:00
darthrubik
4caa28e2cf
Adding jq/bash to dockerfile
2026-05-09 10:06:09 -05:00
darthrubik
9b3ee1e659
Merge pull request 'feature/test-infastructure' ( #8 ) from feature/test-infastructure into master
...
Reviewed-on: https://192.168.8.63/git/darthrubik/drharness/pulls/8
2026-05-09 10:04:19 -05:00
darthrubik
4e947ab8aa
Fixing bug with index being hardcoded in tests
2026-05-09 10:03:19 -05:00
darthrubik
048752c5a4
Simplifying ending a chat from the LLM side in the tests
2026-05-09 09:53:59 -05:00