1
0
mirror of https://github.com/hustcer/deepseek-review.git synced 2026-05-13 05:16:05 +08:00

4 Commits
v1.10 ... v1.11

Author SHA1 Message Date
hustcer
f3ea289921 Bump to v1.11 2025-02-13 19:48:21 +08:00
Justin Ma
4af2f6036b fix: Fix include & exclude pattern matching error (#122)
* fix: Fix include & exclude pattern matching error

* ci skip

* code refactor
2025-02-13 19:44:43 +08:00
hustcer
3dd6b23201 fix: Fix reasoning_content field may not exists error 2025-02-13 15:45:09 +08:00
Justin Ma
c2b26fdc30 fix: Do not show reasoning parts if reviewed by V3 model (#121) 2025-02-13 15:29:10 +08:00
3 changed files with 32 additions and 8 deletions

View File

@@ -1,6 +1,14 @@
# Changelog
All notable changes to this project will be documented in this file.
## [1.11.0] - 2025-02-13
### Bug Fixes
- Do not show reasoning parts if reviewed by DeepSeek V3 model (#121)
- Fix `reasoning_content` field may not exists error
- Fix `include` & `exclude` pattern matching error (#122)
## [1.10.0] - 2025-02-12
### Features

View File

@@ -1,7 +1,7 @@
{
"name": "deepseek-review",
"version": "1.10.0",
"actionVer": "v1.10",
"version": "1.11.0",
"actionVer": "v1.11",
"author": "hustcer",
"license": "MIT",
"github": "https://github.com/hustcer/deepseek-review",

View File

@@ -166,6 +166,7 @@ export def --env deepseek-review [
print $'✖️ Code review failedNo review result returned from ($base_url) ...'
exit $ECODE.SERVER_ERROR
}
let result = if ($reason | is-empty) { $review } else { $result }
if not $is_action {
print $'Code Review Result:'; hr-line; print $result
} else {
@@ -207,11 +208,11 @@ def streaming-output [
if $debug { $last | to json | kv set last-reply }
$last | get -i choices.0.delta | if ($in | is-not-empty) {
let delta = $in
if ($delta.reasoning_content | is-not-empty) { kv set reasoning ((kv get reasoning) + 1) }
if ($delta.reasoning_content? | is-not-empty) { kv set reasoning ((kv get reasoning) + 1) }
if (kv get reasoning) == 1 { print $'(char nl)Reasoning Details:'; hr-line }
if ($delta.content | is-not-empty) { kv set content ((kv get content) + 1) }
if (kv get content) == 1 { print $'(char nl)Review Details:'; hr-line }
print -n ($delta.reasoning_content | default $delta.content)
print -n ($delta.reasoning_content? | default $delta.content)
}
}
@@ -294,7 +295,7 @@ export def get-diff [
print $'(ansi g)Nothing to review.(ansi reset)'; exit $ECODE.SUCCESS
}
let awk_bin = (prepare-awk)
let outdated_awk = $'You may using an (ansi r)outdated awk version(ansi reset), please upgrade to the latest version or use gawk latest instead.'
let outdated_awk = $'If you are using an (ansi r)outdated awk version(ansi reset), please upgrade to the latest version or use gawk latest instead.'
if ($include | is-not-empty) {
let patterns = $include | split row ','
$content = $content | try { ^$awk_bin (generate-include-regex $patterns) } catch { print $outdated_awk; exit $ECODE.OUTDATED }
@@ -408,15 +409,30 @@ export def hr-line [
if $blank_line { char nl | print -n }
}
# Convert glob patterns to regex patterns
# Pass in *.nu directly as a regular expression does not work, because * in
# a regular expression needs to be attached to the previous pattern, the correct
# form should be .* So we should convert each glob pattern to a regex pattern:
# 1. Convert * to .*
# 2. Convert ? to . (optional, as needed)
# 3. Convert / to \/
def glob-to-regex [patterns: list<string>] {
$patterns
| each { |pat|
$pat | str replace "*" ".*" | str replace "?" "." | str replace "/" "\\/"
}
| str join "|"
}
# Generate the awk include regex pattern string for the specified patterns
export def generate-include-regex [patterns: list<string>] {
let pattern = $patterns | each {|pat| $pat | str replace '/' '\/' } | str join '|'
let pattern = glob-to-regex $patterns
$"/^diff --git/{p=/^diff --git a\\/($pattern)/}p"
}
# Generate the awk exclude regex pattern string for the specified patterns
def generate-exclude-regex [patterns: list<string>] {
let pattern = $patterns | each {|pat| $pat | str replace '/' '\/' } | str join '|'
export def generate-exclude-regex [patterns: list<string>] {
let pattern = glob-to-regex $patterns
$"/^diff --git/{p=/^diff --git a\\/($pattern)/}!p"
}