-
Notifications
You must be signed in to change notification settings - Fork 0
Test pr #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
da1z
wants to merge
33
commits into
main
Choose a base branch
from
rewrite
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Test pr #1
Changes from 2 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
72a05cc
Update TypeScript peer dependency to version 5.9.3 and modify tsconfi…
da1z dd5ebef
Add VSCode settings for code formatting and editor configurations
da1z 8ce7091
Update Oxlint configuration, remove outdated files, and enhance CLI s…
da1z 019f04a
Update handleResolve test to support asynchronous execution
da1z 283364e
Add agent and cursor modules with structured output support
da1z c1f910c
Add @da1z/prompt dependency and enhance comment resolution logic
da1z c441604
Refactor comment resolution and enhance templates for clarity
da1z cdda792
Add file system module with readFile functionality
da1z 65bade6
Update readFile method in FileSystem type to support synchronous and …
da1z 9c06b4d
Add skills loading functionality with tests
da1z 2839355
Update dependencies in package.json and bun.lock; add new configurati…
da1z e97789c
effectify config
da1z 95557b8
Update Oxlint configuration to disable relative parent imports and en…
da1z 9f6a1ff
Refactor handleInit to accept context and update config retrieval
da1z 1024b32
Refactor agent functions to return Promises for asynchronous handling
da1z 4fed8af
Add principles section to AGENTS.md for codebase guidance
da1z 2e22c6b
Add CLI entry point and enhance main function for command handling
da1z 270b116
Add parseGitRef function and corresponding tests
da1z 160b2bd
Add glob utility functions for pattern matching
da1z cda5ac3
Update readFile method in FileSystem type to return a Promise<string>…
da1z 26d560b
Refactor CLI structure by moving entry point to a new directory
da1z 062c176
Refactor config schema and enhance configuration handling
da1z 13c3633
Add Context type definition for improved dependency management
da1z 4910242
Refactor handleInit function to use a more specific Context type
da1z b8d84a8
Add reporter functionality and integrate with review process
da1z 5c6263f
Add review target type classification and corresponding tests
da1z 6532598
Add createGitDiffFromFiles function and corresponding tests
da1z 12939d6
Enhance Git and Reporter functionality for review process
da1z 166a92d
Remove main, resolve, and associated test files to streamline the cod…
da1z 2dff56f
Refactor review process and enhance reporter functionality
da1z 44f90cf
Enhance console reporter to include summary of findings by severity
da1z 9cd7cb8
Update agent configuration in handleReview to include model parameter
da1z 97421d3
Update bishop.json to refine bot configuration and introduce review s…
da1z File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| { | ||
| "editor.defaultFormatter": "esbenp.prettier-vscode", | ||
| "typescript.tsdk": "node_modules/typescript/lib", | ||
| "editor.formatOnSave": true, | ||
| "editor.formatOnPaste": true, | ||
| "emmet.showExpandedAbbreviation": "never", | ||
| "[javascript]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[javascriptreact]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[typescript]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[typescriptreact]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[json]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[jsonc]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[yaml]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[html]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[vue]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[vue-html]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[handlebars]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[css]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[scss]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[less]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[graphql]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "[markdown]": { | ||
| "editor.defaultFormatter": "oxc.oxc-vscode" | ||
| }, | ||
| "editor.codeActionsOnSave": { | ||
| "source.fixAll.oxc": "explicit" | ||
| } | ||
| } | ||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,6 +26,6 @@ | |
| "ultracite": "7.2.3" | ||
| }, | ||
| "peerDependencies": { | ||
| "typescript": "^5" | ||
| "typescript": "^5.9.3" | ||
| } | ||
| } | ||
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
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.
Uh oh!
There was an error while loading. Please reload this page.