Repository navigation
feat(skills): integrate conditional task authoring (5by) - #32
Merged
Merged
Conversation
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.
Scope / authority
Integrate only the independently reviewed conditional task-authoring slice for
agentic-devlopment-5by, under current maintainer authorization. One dependency-inseparable logical lot: capability + seven conditional routes + exact eight-capability discovery + documentation and targeted source checks. No main publication, installation, campaigns, unrelated dirty adoption or tracker-parent closure.Independent source review
Reviewer20261114_41 accepted canonical FR-1..FR-6 for source candidate
5a4aacb4c1498349e652dcbde4671f5df8ec4333178a7b99c58ae18f08561856; current original-source recapture SAME. Onlyowned.diffapplied cleanly on actual develop64dd0245d21a1d6f3f3f10d31de9ad6f3731d442; all20 owned file bytes equal accepted source, no adaptation. Reviewer did not construct the patch; builder summary exposure disclosed. No numeric score or behavioral/release acceptance.Verification
Fresh59 focused static/source checks PASS (task_authoring, ticket_authoring, skill_operability, workflow_contracts, production_discovery, phase_final_report, cycle_disclosure); payload inventory and official standalone Skill Creator validation PASS. Before/after capture SAME. Initial discovery run hit /tmp disk quota; fresh /dev/shm TMPDIR rerun passed without source fixes. Full exact-head source CI remains authoritative: contracts, inventory, disposable candidate/verify/smoke. Pages deploy must remain skipped.
Preservation and limits
Original feature checkout HEAD/index/66-path dirty bytes and CLAUDE symlink preserved, no cleanup/stash/reset. Installed plugin untouched. Static checks do not establish live task authoring, host compatibility, reduced effort or release acceptance. Eight guidance copies intentionally retain contained fallback.
Owned paths
AGENTS.mdCONTRIBUTING.mdREADME.mdskills/cycle/SKILL.mdskills/implement/SKILL.mdskills/iterate/SKILL.mdskills/judge/SKILL.mdskills/plan/SKILL.mdskills/research/SKILL.mdskills/task-authoring/SKILL.mdskills/task-authoring/references/authorization-policy.mdskills/task-authoring/references/beads-integration.mdskills/task-authoring/references/ticket-authoring.mdskills/test/SKILL.mdtests/test_phase_final_report.pytests/test_production_discovery.pytests/test_skill_operability.pytests/test_task_authoring.pytests/test_workflow_contracts.pytools/production.py