Skip to content

Integrate openspec skills#212

Open
nhanasi wants to merge 3 commits into
developfrom
feature/openspecinit
Open

Integrate openspec skills#212
nhanasi wants to merge 3 commits into
developfrom
feature/openspecinit

Conversation

@nhanasi

@nhanasi nhanasi commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@nhanasi
nhanasi requested a review from a team as a code owner June 9, 2026 15:34
Copilot AI review requested due to automatic review settings June 9, 2026 15:34
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Code Coverage Summary

                               Total:|84.1%   6263|97.3%  1668|    -      0

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds OpenSpec configuration plus a set of GitHub Copilot “skills” and corresponding /opsx:* prompt templates to support proposing, exploring, applying, and archiving OpenSpec-driven changes in this repo.

Changes:

  • Add openspec/config.yaml setting the default OpenSpec schema (spec-driven).
  • Introduce new OpenSpec skills under .github/skills/ (propose/explore/apply/archive).
  • Add matching /opsx:* prompt files under .github/prompts/ (propose/explore/apply/archive).

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
openspec/config.yaml Adds base OpenSpec config and schema selection for the repository.
.github/skills/openspec-propose/SKILL.md Skill instructions for generating a new change plus proposal/design/tasks artifacts.
.github/skills/openspec-explore/SKILL.md Skill instructions for “explore mode” (requirements clarification / investigation).
.github/skills/openspec-apply-change/SKILL.md Skill instructions for implementing tasks from an OpenSpec change.
.github/skills/openspec-archive-change/SKILL.md Skill instructions for archiving a change, including delta spec sync guidance.
.github/prompts/opsx-propose.prompt.md /opsx:propose prompt instructions aligned to the propose skill workflow.
.github/prompts/opsx-explore.prompt.md /opsx:explore prompt instructions aligned to explore mode.
.github/prompts/opsx-apply.prompt.md /opsx:apply prompt instructions for implementing tasks from a change.
.github/prompts/opsx-archive.prompt.md /opsx:archive prompt instructions for archiving a completed change.

Comment thread .github/prompts/opsx-archive.prompt.md
Comment thread .github/prompts/opsx-archive.prompt.md
Comment thread .github/skills/openspec-archive-change/SKILL.md
Comment thread .github/skills/openspec-archive-change/SKILL.md
Comment thread .github/skills/openspec-propose/SKILL.md
Comment thread .github/skills/openspec-explore/SKILL.md
Comment thread .github/skills/openspec-apply-change/SKILL.md
Comment thread .github/skills/openspec-archive-change/SKILL.md
Comment thread .github/prompts/opsx-apply.prompt.md
Comment thread .github/skills/openspec-apply-change/SKILL.md
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 3 files pending identification.

  • Protex Server Path: /home/blackduck/github/remote_debugger/212/rdkcentral/remote_debugger

  • Commit: f380a8c

Report detail: gist'

@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 3 files pending identification.

  • Protex Server Path: /home/blackduck/github/remote_debugger/212/rdkcentral/remote_debugger

  • Commit: f380a8c

Report detail: gist'

@mhughesacn

Copy link
Copy Markdown

Hi @nhanasi : Where are the new files coming from? There are several code matches, but the tool isn't finding everything. Please confirm the source(s) for all the files which are not original.
At least some are MIT licensed and you need to add a credit to NOTICE:

SKILLS and other prompt material is:
Copyright (c) 2024 OpenSpec Contributors
Licensed under the MIT License

LICENSE in this repo needs to get an MIT licence appended.

I am open to correction on the exact details.
Thank you.

Copilot AI review requested due to automatic review settings June 15, 2026 19:33
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 3 files pending identification.

  • Protex Server Path: /home/blackduck/github/remote_debugger/212/rdkcentral/remote_debugger

  • Commit: ea365b8

Report detail: gist'

@github-actions

Copy link
Copy Markdown

Code Coverage Summary

                               Total:|84.1%   6263|97.3%  1668|    -      0

1 similar comment
@github-actions

Copy link
Copy Markdown

Code Coverage Summary

                               Total:|84.1%   6263|97.3%  1668|    -      0

@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 3 files pending identification.

  • Protex Server Path: /home/blackduck/github/remote_debugger/212/rdkcentral/remote_debugger

  • Commit: ea365b8

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 7 comments.

Comment thread LICENSE

MIT License

Copyright (c) YEAR COPYRIGHT HOLDER
- Dynamic instruction based on current state

**Handle states:**
- If `state: "blocked"` (missing artifacts): show message, suggest using openspec-continue-change
- Dynamic instruction based on current state

**Handle states:**
- If `state: "blocked"` (missing artifacts): show message, suggest using `/opsx:continue`
- If changes needed: "Sync now (recommended)", "Archive without syncing"
- If already synced: "Archive now", "Sync anyway", "Cancel"

If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>"). Proceed to archive regardless of choice.
- Don't block archive on warnings - just inform and confirm
- Preserve .openspec.yaml when moving to archive (it moves with the directory)
- Show clear summary of what happened
- If sync is requested, use openspec-sync-specs approach (agent-driven)
- If changes needed: "Sync now (recommended)", "Archive without syncing"
- If already synced: "Archive now", "Sync anyway", "Cancel"

If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>"). Proceed to archive regardless of choice.
- Don't block archive on warnings - just inform and confirm
- Preserve .openspec.yaml when moving to archive (it moves with the directory)
- Show clear summary of what happened
- If sync is requested, use the Skill tool to invoke `openspec-sync-specs` (agent-driven)
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## WARNING: A Blackduck scan failure has been waived

A prior failure has been upvoted

  • Upvote reason: ok

  • Commit: ea365b8
    '

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants