Skip to content

feat(demos/data-engineering): reorganize into use-cases/ + add general talk track with variants#127

Open
devin-ai-integration[bot] wants to merge 3 commits into
devin/1780958566-demos-reorganizationfrom
devin/1780963538-data-engineering-demos-reorg
Open

feat(demos/data-engineering): reorganize into use-cases/ + add general talk track with variants#127
devin-ai-integration[bot] wants to merge 3 commits into
devin/1780958566-demos-reorganizationfrom
devin/1780963538-data-engineering-demos-reorg

Conversation

@devin-ai-integration

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

Copy link
Copy Markdown
Contributor

Summary

Reorganizes demos/data-engineering/ and adds a source/target-agnostic general migration demo talk track with Cloud and Desktop+Cloud variants.

Structure change:

demos/data-engineering/
├── general.md              # NEW — Cloud variant talk track
├── general.platform.md     # NEW — Desktop + Cloud variant
└── use-cases/
    ├── sas-to-databricks-demo.md   # moved from parent dir
    └── sybase-to-sqlserver-demo.md # moved from parent dir

general.md — Coarse-grained Cloud-only demo covering the universal migration pattern (orient → convert → verify → catch divergences → fan out). Includes source-agnostic prompts in fenced code blocks, verification loop mechanics, scaling via playbooks/child sessions, and links to the specific use-case demos.

general.platform.md — Same arc reframed for Desktop as primary interface: explore legacy estate in a Space, delegate conversions to Cloud, monitor child sessions in Agent Command Center Kanban, review migration PRs via one-click checkout.

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


Open in Devin Review

…l talk track with variants

- Move sas-to-databricks-demo.md and sybase-to-sqlserver-demo.md into use-cases/
- Add general.md: Cloud-only coarse-grained data migration demo talk track
- Add general.platform.md: Desktop + Cloud variant with Agent Command Center
@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 5 potential issues.

View 1 additional finding in Devin Review.

Open in Devin Review

Comment thread demos/data-engineering/general.md Outdated
Comment thread demos/data-engineering/general.platform.md Outdated
Comment thread demos/data-engineering/general.md
Comment on lines +99 to +105
```
Using the <source-repo> repo, give me a map of the legacy estate:
tables, programs/procedures, views, macros/functions, and batch
orchestration. For each object, identify platform-specific constructs
that need conversion to <target-platform>. Output a structured
inventory as a Markdown table.
```

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.

🚩 Prompt specificity uses template placeholders intentionally

The prompts in both files use generic placeholders like <source-repo>, <target-platform>, <object-name> rather than concrete repo names and file paths. The AGENTS.md rule says "Prompts must include repo names, file paths, and expected output format." However, these are general methodology documents that explicitly link to specific worked examples (use-cases/sas-to-databricks-demo.md, use-cases/sybase-to-sqlserver-demo.md) at the bottom. The placeholders are intentional template syntax for a cross-platform guide, not vague prompts. The specific worked examples are expected to contain concrete paths.

Open in Devin Review

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

@@ -0,0 +1,181 @@
# Data Platform Migration with Devin — General Demo

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.

🚩 Substantial preamble before first actionable prompt may tension with demos/AGENTS.md guidance

The demos/AGENTS.md says "Lead straight into the guide with prompts and user instructions. Keep preamble minimal — no long orientation before the first step." In general.md, the first actionable prompt doesn't appear until line 100, after ~67 lines of pattern explanation and verification loop description. In general.platform.md, the first prompt is at line 87. These are general methodology/overview documents rather than step-by-step demo walkthroughs, and they link to specific worked examples for the actual hands-on path. Whether they need to follow the "minimal preamble" rule as strictly as a concrete demo is a judgment call for the reviewer.

Open in Devin Review

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

- Remove 'open a PR' phrasing from fan-out prompts per AGENTS.md rules
- Update catalog/repos.md links to use-cases/ subdirectory paths

@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.

⚠️ 1 issue in files not directly in the diff

⚠️ Broken relative link after file rename — resolves to non-existent path (demos/data-engineering/use-cases/sas-to-databricks-demo.md:397)

The file demos/data-engineering/sas-to-databricks-demo.md was moved one directory deeper to demos/data-engineering/use-cases/sas-to-databricks-demo.md, but the relative link at line 397 (../../modules/data-engineering/sas-migration-analysis.md) was not updated. From the new location, ../../ resolves to demos/ (not the repo root), so the link points to the non-existent path demos/modules/data-engineering/sas-migration-analysis.md. The correct relative path should be ../../../modules/data-engineering/sas-migration-analysis.md.

View 2 additional findings in Devin Review.

Open in Devin Review

(source analysis → target mapping → produce PR → programmatic verification → human
review → refine) described in
[`modules/data-engineering/sas-migration-analysis.md`](../../modules/data-engineering/sas-migration-analysis.md)
[`modules/data-engineering/sas-migration-analysis.md`](../../../modules/data-engineering/sas-migration-analysis.md)

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.

Fixed in 700bf2f — updated the relative link from ../../ to ../../../ to account for the extra directory depth after the move to use-cases/. Also confirmed the Sybase demo has no relative links that need updating.

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