feat(workshops/data-source-migration): add platform story coverage + delivery variants#112
Conversation
…delivery variants
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
There was a problem hiding this comment.
🚩 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.
Was this helpful? React with 👍 or 👎 to provide feedback.
There was a problem hiding this comment.
🚩 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.
Was this helpful? React with 👍 or 👎 to provide feedback.
There was a problem hiding this comment.
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.
Summary
Adds platform story coverage to the data-source-migration workshop with two changes:
README.md(cloud-agent baseline) — additive updates only:README.platform.md## Platform Contextsection after Overview: explains this is a Devin Cloud / web-app delivery, with a> **Tip:**callout pointing to Desktop + CLI alternatives> **Desktop tip:**after Step 1 (one-click PR checkout) and Step 4 (Agent Command Center)> **CLI alternative:**after Step 2 (local exploration withdevin)README.platform.md(Desktop + Cloud variant) — new file:## Platform Contextsection explaining Desktop + Cloud deliveryLink to Devin session: https://partner-workshops.devinenterprise.com/sessions/924acf3e81584b10a531d4ef7aa10abc
Requested by: @bsmitches