Fix imports #41
Fix imports #41
Conversation
Resolve merge conflict in package.json Keep version 1.13.0 and framework description Add library exports epic for explicit imports
Use concise format with Given/Should requirements Remove verbose sections and task numbering
Add TypeScript validation for .d.ts files Add note about automated export tests
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new epic task to address the library export strategy and increments the package version to 1.13.0. The epic outlines a plan to move from barrel exports to explicit import paths for better tree-shaking and to resolve the issue where package.json references a non-existent root index.js file.
- New epic document defining the library exports refactoring strategy
- Version bump from 1.12.0 to 1.13.0 in preparation for the changes
- Package-lock.json updated with peer dependency markers for consistency
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tasks/library-exports-epic.md | New epic document outlining the plan to fix library exports using explicit paths instead of barrel exports |
| package.json | Version bumped from 1.12.0 to 1.13.0 |
| package-lock.json | Version updated to match package.json, and peer dependency flags added |
Comments suppressed due to low confidence (1)
tasks/library-exports-epic.md:1
- The epic mentions that documentation shows imports from
'aidd'that don't work (line 10), and the task on line 66 states to change'aidd'to'aidd/asyncPipe'. However, line 66 incorrectly references changing'aidd'when the actual existing import pattern in docs/server/README.md isimport { asyncPipe } from 'aidd';(which imports a named export). The requirement should clarify changing fromimport { asyncPipe } from 'aidd'toimport asyncPipe from 'aidd/asyncPipe'or similar, as the import style will change from named to default export when using explicit paths.
# Fix Library Exports Epic
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Document development workflow using /task, /review, /execute Add export conventions favoring named exports Update README to reference CONTRIBUTING.md
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Move development workflow to README getting started CONTRIBUTING now references README instead of duplicating Remove redundant examples and verbose content Update TOC
Updated the process steps for project planning and execution in the README.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ## Questions or Issues? | ||
|
|
||
| - Check existing [issues](https://github.com/paralleldrive/aidd/issues) |
There was a problem hiding this comment.
The GitHub issues link references paralleldrive/aidd but the repository URL in package.json is paralleldrive/sudolang.ai. This should be updated to https://github.com/paralleldrive/sudolang.ai/issues to match the actual repository.
| - Check existing [issues](https://github.com/paralleldrive/aidd/issues) | |
| - Check existing [issues](https://github.com/paralleldrive/sudolang.ai/issues) |
Note
Adds a CONTRIBUTING guide, documents a Development Workflow in the README, introduces a library exports task epic, and bumps the package version to 1.13.0.
CONTRIBUTING.mdwith code style, testing, commit, and PR guidelines.CONTRIBUTING.md.tasks/library-exports-epic.mdoutlining explicit path export conventions and related work.1.12.0to1.13.0inpackage.json(and lockfile).Written by Cursor Bugbot for commit 46c3bff. This will update automatically on new commits. Configure here.