Repository navigation
feat(reg-intel): align discovery model and date validation with spec (THQ-356) - #537
Merged
Merged
Conversation
Task transitrix-hq#356 — fix reg-intel documentation and implementation to match 14-codex.md §3.7 spec: - Discovery now reads from operations/config/scan-sources.yaml (the centralised watch-list), not by walking codex/ directory. Updated SKILL.md, CLI README, and package.json descriptions to reflect this. - Codex watch-list fixture creates scan-sources.yaml per the spec schema. When a source is static (monitoring_needed: false), its monitor_instead entries are surfaced as separate scan targets. - Added date validation to reject impossible dates (e.g., 2026-02-31) and malformed ISO-8601 dates (e.g., 2026-6-01). Both addFrequency and isDue now validate date format and reject non-dates rather than silently comparing them lexicographically. - Added test Part M to cover date validation edge cases: impossible dates, malformed ISO dates, and non-date strings. - Created GitHub Actions workflow (.github/workflows/reg-intel-skill-test.yml) to run the integrity test on PRs touching reg-intel skill or CLI. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Signed-off-by: transitrix <automation@transitrix.com>
1 of 2 tasks
vkgeorgia
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Task THQ-356 — fix the reg-intel skill and CLI to properly implement the codex discovery model and date handling as specified in 14-codex.md §3.7.
Changes
Discovery model alignment: Updated documentation and implementation to read the source registry from
operations/config/scan-sources.yaml(the centralised watch-list), not by walking thecodex/directory. This makes discovery explicit and allows future non-codex connector types (APIs, Confluence, etc.).Test fixture: Created
scan-sources.yamlwith proper schema. Static sources (monitoring_needed: false) are now handled correctly — theirmonitor_insteadentries are surfaced as separate scan targets.Date validation: Added proper date validation to reject:
isDue(no longer silently returns false via string comparison)CI workflow: Added
.github/workflows/reg-intel-skill-test.ymlto run the integrity test on PRs that touch the skill or CLI.Test Plan
python transitrix/skills/reg-intel/tests/test_reg_intel_integrity.pypasses all checks (Parts A–M)🤖 Generated with Claude Code