Skip to content

feat(workshops/quality-engineering-security): add platform story coverage + delivery variants#116

Open
devin-ai-integration[bot] wants to merge 1 commit into
devin/1780546598-platform-story-coveragefrom
devin/1780845498-quality-engineering-security-platform-variants
Open

feat(workshops/quality-engineering-security): add platform story coverage + delivery variants#116
devin-ai-integration[bot] wants to merge 1 commit into
devin/1780546598-platform-story-coveragefrom
devin/1780845498-quality-engineering-security-platform-variants

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds platform story coverage to the quality-engineering-security workshop with two deliverables:

README.md (Cloud variant — updated):

  • Added ## Platform Context section after the Abstract explaining this is a Cloud-via-web-app workshop, with a tip callout linking to the Desktop variant
  • Added cross-surface callout blocks throughout existing labs:
    • Lab 1: Desktop tip (one-click PR checkout)
    • Lab 2: CLI alternative (local code exploration)
    • Lab 3: Desktop tip (Agent Command Center for parallel SAST monitoring)
    • Lab 4: Desktop tip (one-click checkout for CI workflow YAML review)
  • Added Agent Command Center bullet to "Getting the Most from This Workshop"
  • Added variant header note linking to README.platform.md
  • No changes to existing lab structure, track numbering, prompts, repo references, or timing

README.platform.md (Desktop + Cloud variant — new):

  • Same lab structure (4 labs, same tracks, same target outcomes)
  • Each lab reframed with a "Desktop Workflow" section: explore locally with Cascade/Devin Local → delegate to Cloud → monitor in Agent Command Center → review PRs with one-click checkout
  • Overview and Abstract rewritten for Desktop + Cloud delivery surface
  • Platform Context section describes the 5-step Desktop workflow (Space → local explore → delegate → monitor → review)
  • References ACP and multi-agent extensibility
  • Devin Features Checklist updated to include Desktop-specific activities (Spaces, Agent Command Center, one-click checkout, parallel Cloud sessions)

Link to Devin session: https://partner-workshops.devinenterprise.com/sessions/75307981bd5943ad8273e5b352ba1088
Requested by: @bsmitches


Open in Devin Review

…rage + delivery variants

- Add Platform Context section and cross-surface callout blocks to README.md (Cloud variant)
- Add variant note linking to README.platform.md
- Add Agent Command Center tip to Getting the Most section
- Create README.platform.md (Desktop + Cloud variant) with Desktop-first workflow
  for each lab: explore locally with Cascade, delegate to Cloud, monitor in
  Agent Command Center, review PRs with one-click checkout
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

View 3 additional findings in Devin Review.

Open in Devin Review

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🚩 Post-Event section also exists pre-PR in README.md (same pattern)

The Post-Event checklist flagged as a bug in README.platform.md:217-222 also exists at README.md:178-183 (unchanged by this PR). It's a systemic pattern across all workshop READMEs (enterprise-security-automation/README.md:201, cobol-modernization/README.md:278, etc.). While the rule violation is clear, this is a repo-wide convention that would need a coordinated cleanup across all workshop files, not just this PR.

(Refers to lines 178-183)

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Comment on lines +217 to +222
## Post-Event

- [ ] Collect participant feedback
- [ ] Archive any event-specific branches
- [ ] Update challenge modules if issues were discovered
- [ ] Share session recordings/artifacts with participants

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🟡 Facilitator-only post-event checklist in attendee-facing repo

The new README.platform.md includes a ## Post-Event section (lines 217–222) containing a facilitator checklist ("Collect participant feedback", "Archive any event-specific branches", etc.). REVIEW.md explicitly states: "Flag any facilitator-only content (MCP setup, presales positioning, timing guides, pacing tips for facilitators, post-event checklists). These belong in the operator repo." Attendees do not collect feedback or archive branches — this section is directed at facilitators and should be moved to the operator repo.

Suggested change
## Post-Event
- [ ] Collect participant feedback
- [ ] Archive any event-specific branches
- [ ] Update challenge modules if issues were discovered
- [ ] Share session recordings/artifacts with participants
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Acknowledged — the ## Post-Event section is indeed facilitator-facing. However, it's a pre-existing pattern carried forward from the base README.md (lines 161–166, unchanged by this PR) for structural consistency across variants. As the second comment notes, this same pattern exists across enterprise-security-automation/README.md, cobol-modernization/README.md, etc.

Removing it only from README.platform.md would create an inconsistency between variants. A repo-wide cleanup across all workshop READMEs would be the right approach — happy to do that as a separate PR if desired.

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.

1 participant