Repository navigation
docs(boost): sync RHDHPLAN-1508 OpenSpec from feature refinement - #5414
johnmcollier merged 2 commits into
Conversation
Signed-off-by: John Collier <jcollier@redhat.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5414 +/- ##
=======================================
Coverage 65.35% 65.35%
=======================================
Files 2758 2758
Lines 110579 110579
Branches 31085 31085
=======================================
Hits 72264 72264
Misses 37739 37739
Partials 576 576
*This pull request uses carry forward flags. Click here to find out more. Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
Code Review by QodoView medium (2)
1. Readers see conflicting usage rules
|
PR Summary by QodoSync RHDHPLAN-1508 RBAC OpenSpec and archive the earlier draft
AI Description
Diagram
High-Level Assessment
Files changed (32)
|
|
Important The |
gabemontero
left a comment
gabemontero
left a comment
There was a problem hiding this comment.
My scan of the new openspec content came up with no issues to report @johnmcollier - lgtm
I would encourage you to be more aggressive with the "legacy content". In particular:
- just delete all the RHDHPLAN-1508 openspec content currently under
workspaces/boost/RHDH-2-1-Legacy-Content/Pre-RHDHPLAN-15xx-Content... AI and I see that list as the 2 top level 1508 docs, RHDHPLAN-1508-feasibility-report.md and RHDHPLAN-1508-frontend-coverage.md; then, I would just delete the security-safety-governance openspecs - then there is the code generated with the prior openspecs
plugins/ai-catalog-common/src/permissions.ts
plugins/ai-catalog-common/src/index.ts
plugins/ai-catalog-common/src/index.test.ts
plugins/boost-backend/src/ai-catalog/CatalogAssetLoader.ts
plugins/boost-backend/src/ai-catalog/CatalogAssetLoader.test.ts
plugins/boost-backend/src/ai-catalog/routes.ts
plugins/boost-backend/src/ai-catalog/routes.test.ts
plugins/boost-backend/src/ai-catalog/rules.ts
plugins/boost-backend/src/ai-catalog/rules.test.ts
plugins/boost-backend/src/ai-catalog/index.ts
plugins/boost-backend/src/plugin.ts
plugins/boost-backend/src/index.ts
plugins/boost-backend/src/ingestion/routes.ts
plugins/boost-backend/src/ingestion/routes.test.ts
if you think you'll reference those in any way when you re-generate code with you new openspecs, then keep them for that. If you are going to generate code solely with the new openspecs, go ahead and delete this code under workspaces/boost/RHDH-2-1-Legacy-Content/Pre-RHDHPLAN-15xx-Content with this PR
then lastly, I see you requested a review from @rohitkrai03 .... definitely get his take on this PR, make sure the frontend is OK with where we are landing with you changes, before we merge.
thanks
Signed-off-by: John Collier <jcollier@redhat.com>
|
gabemontero
left a comment
gabemontero
left a comment
There was a problem hiding this comment.
lgtm @johnmcollier
per my previous comment, let's get a review form @rohitkrai03 on this if at all possible before merging
…hat-developer#5414) * docs(boost): sync RHDHPLAN-1508 OpenSpec Signed-off-by: John Collier <jcollier@redhat.com> * docs(boost): remove obsolete 1508 legacy artifacts Signed-off-by: John Collier <jcollier@redhat.com> --------- Signed-off-by: John Collier <jcollier@redhat.com>
Summary
Sync the merged RHDHPLAN-1508 OpenSpec change from
redhat-developer/agentic-feature-refinement(source commitba2dbf1e141d7ded6a8ebe0a92eb5115e67f0cc8) into the Boost workspace. The six capability specs and supporting artifacts are copied from that change. The OpenSpec metadata uses Boost'sspec-drivenschema, and the source README and two Markdown files receive workspace provenance/formatting adjustments.Archive the earlier
ai-catalog-asset-governancedraft so there is one active 1508 governance change. Remove the legacy 1508 feasibility and frontend reports, the old security OpenSpec area, and selected generated common/backend snapshot files. Update workspace guidance and historical references to match the cleanup. No active runtime package behavior changes.Validation
yarn openspec:validate— 18/18 active Boost specs and changes passedyarn prettier:check— passedgit diff --check— passed✔️ Checklist