oblachno-oss/devx
Public Access
3
0
Fork 0
Code Issues 95 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

DEVX-163: Add vikunja-tasks skill and skill validation tests, fix create-task docs #323

Merged
kireto merged 2 commits from DEVX-163-skills-vikunja-validation into master 2026-09-18 22:47:33 +00:00
Conversation 2 Commits 2 Files Changed 7
+292 -21
kireto commented 2026-09-18 22:44:06 +00:00
Member
Copy Link
Copy Source
No description provided.
kireto added 1 commit 2026-09-18 22:44:07 +00:00
docs: add vikunja-tasks skill and skill validation tests, fix create-task docs
CI / validate (pull_request) Failing after 47s
Details
CI / auto-merge (pull_request) Skipped
Details
758dab9ddc 
- Add vikunja-tasks: task create/query/close lifecycle
- New tests/unit/test_skills_validation.py: structure, make-target,
  file-ref checks + existence tests for all 8 skills; devx.mak targets
  resolved from src/devx/make/ (editable install)
- devx-workflow/spec-driven-development/testing-and-debugging: add
  When to Invoke + Prerequisites; fix broken create-task syntax
- Preserve colliding spec as DEVX-163-run-push-stdout-historical

Implements: REQ-1 REQ-2 REQ-3
emil added 1 commit 2026-09-18 22:46:36 +00:00
docs: conform spec to required template
CI / validate (pull_request) Successful in 45s
Details
CI / auto-merge (pull_request) Successful in 11s
Details
817c6929f8
emil approved these changes 2026-09-18 22:47:30 +00:00
emil left a comment
Owner
Copy Link
Copy Source

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
kireto merged commit b2ac1cd06c into master 2026-09-18 22:47:33 +00:00
kireto deleted branch DEVX-163-skills-vikunja-validation 2026-09-18 22:47:33 +00:00
emil reviewed 2026-09-18 22:50:25 +00:00
emil left a comment
Owner
Copy Link
Copy Source

Deep Review Summary

  • Files reviewed: 7 (1 new skill, test file, 3 skill fixes, spec + historical)
  • Issues found: 4 — all auto-fixed (missing required sections, devx.mak target resolution for editable install, spec template, create-task syntax)
  • Coverage: new tests/unit/test_skills_validation.py — 10 tests covering all 8 skills

Outcome: Ready to merge — docs/skills only.

## Deep Review Summary - **Files reviewed:** 7 (1 new skill, test file, 3 skill fixes, spec + historical) - **Issues found:** 4 — all auto-fixed (missing required sections, devx.mak target resolution for editable install, spec template, create-task syntax) - **Coverage:** new `tests/unit/test_skills_validation.py` — 10 tests covering all 8 skills **Outcome:** Ready to merge — docs/skills only.
emil added the ready-to-merge label 2026-09-18 22:50:47 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
emil
Labels
Clear labels
bug
Something is not working
 ci-improvement
CI workflow improvements
 doc-improvement
Documentation improvements
 feedback
Issues from subagent or agent feedback
 ready-to-merge
refactoring
Large refactoring PR (bypasses PR size check)
 tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels ready-to-merge
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) emo kireto
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#323