Repository navigation
[FEAT]: Add RAMPART onboarding skill - #195
Merged
Nina Chikanov (nina-msft) merged 5 commits intoOct 13, 2026
Merged
Nina Chikanov (nina-msft) merged 5 commits into
Nina Chikanov (nina-msft) merged 5 commits into
Conversation
Package a portable onboarding and learning skill that guides application-specific setup, evidence agreements, first tests, results, and relevant follow-up coverage. Add installation documentation, navigation links, and regression coverage for the skill contract and offline examples. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Nina Chikanov (nina-msft)
changed the title
Nina Chikanov (nina-msft)
marked this pull request as ready for review
October 6, 2026 23:02
Copilot started reviewing on behalf of
Spencer Schoenberg (spencrr)
October 6, 2026 23:03
View session
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The skill, documentation, packaging path, and regression coverage are consistent with the existing framework APIs.
Review effort: Balanced
Findings: None
What changed in this PR
Adds RAMPARTner, a portable onboarding skill that guides developers through creating application-specific RAMPART safety tests.
Changes:
- Adds the self-contained onboarding skill and offline smoke-test example.
- Documents installation and discovery across supported coding assistants.
- Adds regression tests for metadata, guidance, and example outcomes.
| File | Description |
|---|---|
rampart/skills/rampartner/SKILL.md |
Defines the onboarding workflow and guidance. |
tests/unit/test_onboarding_skill.py |
Validates metadata, content, and examples. |
docs/getting-started/onboarding-skill.md |
Documents installation and usage. |
docs/getting-started/index.md |
Links the onboarding guide. |
docs/getting-started/quickstart.md |
Promotes guided onboarding. |
README.md |
Introduces RAMPARTner. |
mkdocs.yml |
Adds the guide to navigation. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Remove the standalone smoke test and fixed Helpdesk example. Keep guidance tied to the user's workflow and offer a choice between practicing with the current test and moving to another case. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Use the PR merge commit's first parent instead of the event payload's potentially stale base SHA. Preserve content-only trace validation for pushes to main. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Nina Chikanov (nina-msft)
enabled auto-merge (squash)
October 12, 2026 19:31
Bashir Partovi (bashirpartovi)
approved these changes
Oct 13, 2026
Bashir Partovi (bashirpartovi)
left a comment
Bashir Partovi (bashirpartovi)
left a comment
Contributor
There was a problem hiding this comment.
Such a great addition, thanks Nina Chikanov (@nina-msft)
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.
Description
Developers need help turning their own application's behavior and risks into meaningful RAMPART tests. This adds RAMPARTner, a portable onboarding and learning skill that introduces itself with "Howdy! I'm your RAMPARTner" and guides users through that process in their coding assistant.
mainor the latest release), application-specific integration, agreement on safety claims and evidence, and authorized first-test execution.The skill provides instructions, not a separate agent runtime or enforced state machine. Execution remains subject to the host assistant's tools and permissions. Its focus is onboarding and learning, not ongoing CI or suite management.
CI correction: The trace-compatibility gate now compares against the tested PR merge commit's first parent instead of the event payload's potentially stale base SHA. This prevents upstream trace-contract changes from being incorrectly attributed to the PR. Pushes to
mainretain content-only validation.Breaking changes
None.
Checklist
pre-commit run --all-filespassesNo new automated tests are included in the final diff.
CI passed Lint & Type Check and the existing test suites on Python 3.11, 3.12, 3.13, and 3.14 for commit
255c7c8. The full all-files pre-commit command was not run.