Skip to content

feat(workshops/data-source-migration): add platform story coverage + delivery variants#112

Open
devin-ai-integration[bot] wants to merge 2 commits into
devin/1780546598-platform-story-coveragefrom
devin/1780845503-data-source-migration-platform-variants
Open

feat(workshops/data-source-migration): add platform story coverage + delivery variants#112
devin-ai-integration[bot] wants to merge 2 commits into
devin/1780546598-platform-story-coveragefrom
devin/1780845503-data-source-migration-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 data-source-migration workshop with two changes:

README.md (cloud-agent baseline) — additive updates only:

  • Variant header linking to README.platform.md
  • ## Platform Context section after Overview: explains this is a Devin Cloud / web-app delivery, with a > **Tip:** callout pointing to Desktop + CLI alternatives
  • Cross-surface callout blocks placed at natural points in the existing lab:
    • > **Desktop tip:** after Step 1 (one-click PR checkout) and Step 4 (Agent Command Center)
    • > **CLI alternative:** after Step 2 (local exploration with devin)
  • New bullet in tips section about tracking parallel sessions via Agent Command Center
  • No existing lab structure, prompts, timing, or content altered

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

  • Same lab structure (Lab 1, same repo, same objective, same target outcomes)
  • Workflow reframed for Desktop as primary interface:
    1. Create a Space for the workshop
    2. Explore code locally with Cascade / Devin Local
    3. Delegate implementation to Devin Cloud from Desktop
    4. Monitor in Agent Command Center (Kanban view)
    5. Review PRs with one-click checkout in-editor
  • ## Platform Context section explaining Desktop + Cloud delivery
  • ACP callout noting third-party agent extensibility
  • Two additional Key Takeaways on the Desktop command center and platform continuum

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


Open in Devin Review

@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 1 additional finding 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.

🚩 Pre-existing prompt in README.md is not line-wrapped at ~80 chars

The prompt in workshops/data-source-migration/README.md:56-62 has lines well over 80 characters (e.g., line 58 is ~200+ chars). AGENTS.md requires "Line-wrap prompts at ~80 chars for readability." The same prompt in the new workshops/data-source-migration/README.platform.md:77-92 IS properly line-wrapped. This is a pre-existing issue — the PR didn't modify the prompt lines in README.md — but the inconsistency between the two variants is notable. The prompt lines fall outside the changed diff hunks, so this cannot be reported as a bug per the review constraints.

Open in Devin Review

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

Comment thread workshops/data-source-migration/README.md Outdated

@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 1 new potential issue.

View 1 additional finding 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.

🚩 No automation narrative (webhooks, scheduled sessions, child sessions) in either variant

AGENTS.md states "Every module and workshop should incorporate applicable principles from shared/general-themes/" including "automation narrative: webhooks, scheduled sessions, child sessions for divide-and-conquer." Neither the Cloud-only README.md nor the new README.platform.md mentions these concepts. This is pre-existing for README.md and carried over to the new platform variant. Whether automation narrative is "applicable" to a data source migration workshop is a judgment call — it's plausible that scheduled sessions or child sessions could be relevant for large-scale migration scenarios. This is a medium-priority general themes integration item per REVIEW.md.

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 — this is a pre-existing gap in the base workshop content, not introduced by this PR. The scope here was limited to adding platform context sections and cross-surface callouts without altering existing lab content. Adding automation narrative (scheduled sessions, child agents for large-scale migrations) would be a good follow-up but falls outside this PR's scope.

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