Landing: current comparison; README links to pages instead of embedding map - #119
Merged
Conversation
…s to pages, drops map image
Reviewer's guide (collapsed on small PRs)Reviewer's GuideLanding page content is updated to reflect v0.3.x workflow and a more accurate competitive landscape, while the README now links to the external overview/comparison page instead of embedding the harness map image. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The comparison table relies heavily on color and short "y/n" labels; consider adding more descriptive text or icons so the differences are clearer and more accessible to screen readers and color-blind users.
- Inline styles are used repeatedly in the landscape comparison table (e.g., for gold text and padding); extracting these into CSS classes would simplify future maintenance and keep the HTML markup cleaner.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The comparison table relies heavily on color and short "y/n" labels; consider adding more descriptive text or icons so the differences are clearer and more accessible to screen readers and color-blind users.
- Inline styles are used repeatedly in the landscape comparison table (e.g., for gold text and padding); extracting these into CSS classes would simplify future maintenance and keep the HTML markup cleaner.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c3554910cb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <div class="path-label" style="background:rgba(245,200,106,.12); color:var(--gold);">Private config</div> | ||
| <h3>Set up your home</h3> | ||
| <p class="desc">Detect harnesses, import existing content by copy, and run the first native sync.</p> | ||
| <p class="desc">Detect harnesses, import existing content by copy, offer git init, and run the first native sync. Removals are previewed per harness before anything is touched.</p> |
There was a problem hiding this comment.
setup does not wait for this removal preview before touching native configuration: runSetup calls patchAgentConfig for detected harnesses and the memory hook migration/cleanup helpers before it enables ConfirmRemovals and enters runSync, where the destructive-action preview runs. For users with Hermes/Amp config or legacy memory hooks, following this page can therefore modify harness config before the promised preview; please limit the claim to removal/overwrite actions or move the preview earlier.
Useful? React with 👍 / 👎.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
dotagents skill new.Summary by Sourcery
Update the landing page and README to reflect current comparisons and getting-started flow, emphasizing dotagents’ user-level scope and directing users to the hosted overview site.
New Features:
Enhancements: