docs: Reposition around data engineering harness - #2399
Conversation
There was a problem hiding this comment.
Needs improvement — two user-facing documentation paths are broken.
[RISK ASSESSMENT]
MEDIUM: this documentation change affects first-run onboarding and the recovery link printed after a failed CLI test.
Verdict: COMMENT; address the two compatibility regressions before merging. Architectural insight: published documentation routes are public interfaces consumed by both readers and CLI output.
Improve this review? If feedback seems incorrect or irrelevant, update the repository's .agents/skills/custom-codereview-guide.md (with the /codereview trigger), then re-request review. The reviewer reads the guide from the PR head.
Resolve with AI? Install the iterate skill and run /iterate.
Was this review helpful? React with 👍 or 👎.
This review was generated by an AI agent (OpenHands) on behalf of the user.
Was this automated review useful? React with 👍 or 👎 to this review to help us measure review quality.
Workflow run: https://github.com/DataSQRL/sqrl/actions/runs/35935796951
2947fe6 to
1392981
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2399 +/- ##
============================================
- Coverage 71.34% 71.32% -0.02%
Complexity 1900 1900
============================================
Files 677 677
Lines 20356 20356
Branches 2553 2553
============================================
- Hits 14522 14519 -3
Misses 4528 4528
- Partials 1306 1309 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
# Conflicts: # agent/README.md
# Conflicts: # documentation/src/pages/index.tsx
87cf0fa to
625b6fd
Compare
This implements the repositioning discussed in:
https://docs.google.com/document/d/17q5ooVeT73150bGJg-rWGvKsFuib8nwvaB2daz8WJAU
It is missing the elements listed under "Proposal". Those still need to be implemented.