fix: #75 twist 関数で文字列の前方一致していたところを、キーの前方一致判定に変更 #76

mew-ton · 2023-12-01T02:42:20Z

Summary by CodeRabbit

新機能
- アプリに検索機能を追加しました。
バグ修正
- "json-origami"モジュールの問題 twist method does not produce normal results for forward matching key combinations #75 に対する修正を行いました。
機能改善
- データのキーに共通のプレフィックスがある場合に正しくデータを操作するための新しいテストケースを追加しました。
- キーのプレフィックスマッチングのロジックを改善しました。

changeset-bot · 2023-12-01T02:42:24Z

🦋 Changeset detected

Latest commit: c2067b7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
json-origami	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

coderabbitai · 2023-12-01T02:42:29Z

Walkthrough

変更の概要は、プロジェクト全体の設定とコントロールフローに関するものです。VS Codeのワークスペース設定にTypeScriptのパスが追加され、twist関数のロジックが改善されました。また、新しいユーティリティ関数includesKeyが導入され、より効率的なキー検索が可能になりました。テストケースも更新され、新しいロジックが正しく機能することが確認されています。

Changes

ファイル	変更概要
`.vscode/project.code-workspace` `src/twist.ts`	"folders"キーにネスティングレベルを追加し、"settings"キーに"typescript.tsdk"プロパティを追加。 `includesKey`関数をインポートし、`twist`関数内のロジックを変更
`test/.../twist.spec.ts`	共通プレフィックスを持つキーマップデータの正しい扱いを確認する新しいテストケースを追加

🐰✨
コードの森を跳ねる小さな兎よ
新しい葉を見つけたんだね
型の木々、輝くTSの道
今日もまた、バグを追い越せ

Tips

Chat with CodeRabbit Bot (`@coderabbitai`)

If you reply to a review comment from CodeRabbit, the bot will automatically respond.
To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
Note: Review comments are made on code diffs or files, not on the PR overview.
Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Commands (invoked as PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
The JSON schema for the configuration file is available here.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

mew-ton · 2023-12-01T02:42:28Z

.vscode/project.code-workspace

+  ],
+  "settings": {
+    "typescript.tsdk": "node_modules/typescript/lib"
+  }


coderabbitai

Review Status

Actionable comments generated: 1

Configuration used: CodeRabbit UI

Commits

Files that changed from the base of the PR and between fcac62f and 1bcd338.

Files selected for processing (3)

.vscode/project.code-workspace (1 hunks)
src/twist.ts (2 hunks)
test/twist.spec.ts (1 hunks)

Additional comments: 3

.vscode/project.code-workspace (1)

1-9: 設定ファイルの変更点は問題ないように見えますが、プロジェクトの他の部分に影響を与えないか確認してください。特に、TypeScriptの設定変更はプロジェクトのビルドや実行に影響を与える可能性があります。

src/twist.ts (1)

3-4: 新しいユーティリティ関数includesKeyのインポート文が正しく追加されました。

test/twist.spec.ts (1)

251-285: 新しいテストケースが追加され、マップデータのキーが共通のプレフィックスを持つ場合にデータが正しく変換されることを確認しています。この変更は、要約に記載されている内容と一致しています。

coderabbitai · 2023-12-01T02:44:20Z

src/twist.ts

  const twisted = Object.fromEntries(
    Object.entries(folded).map(([key, value]) => {
-      const found = Object.keys(moveMap).find((k) => key.startsWith(k))
+      const found = Object.keys(moveMap).find((k) => includesKey(key, k, option))

      if (found) {
        const newKey = key.replace(found, moveMap[found]!)


includesKey関数の使用方法が間違っています。Object.keys(moveMap).findを使う代わりに、直接includesKeyを使ってマッチするキーを見つけるべきです。

- const found = Object.keys(moveMap).find((k) => includesKey(key, k, option)) + const found = includesKey(key, Object.keys(moveMap), option)

Committable suggestion

❗ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change

const twisted = Object.fromEntries(

Object.entries(folded).map(([key, value]) => {

const found = Object.keys(moveMap).find((k) => key.startsWith(k))

const found = Object.keys(moveMap).find((k) => includesKey(key, k, option))

if (found) {

const newKey = key.replace(found, moveMap[found]!)

const twisted = Object.fromEntries(

Object.entries(folded).map(([key, value]) => {

const found = includesKey(key, Object.keys(moveMap), option)

if (found) {

const newKey = key.replace(found, moveMap[found]!)

間違ってないです。

@mew-ton 承知しました。現在の実装を維持することにします。もし他に何か手伝えることがあれば、お知らせください。

coderabbitai

Review Status

Actionable comments generated: 0

Configuration used: CodeRabbit UI

Commits

Files that changed from the base of the PR and between 1bcd338 and c2067b7.

Files selected for processing (1)

.changeset/khaki-sheep-roll.md (1 hunks)

Additional comments: 1

.changeset/khaki-sheep-roll.md (1)

1-5: 変更セットの記述が適切であり、twist 関数のキーの前方一致判定への変更を正確に反映しています。

skadota

LGTM！

fix: #75 twist 関数で文字列の前方一致していたところを、キーの前方一致判定に変更

1bcd338

mew-ton commented Dec 1, 2023

View reviewed changes

.vscode/project.code-workspace

],

"settings": {

"typescript.tsdk": "node_modules/typescript/lib"

}

Copy link

Contributor Author

mew-ton Dec 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ついで

Create khaki-sheep-roll.md

c2067b7

coderabbitai bot reviewed Dec 1, 2023

View reviewed changes

skadota approved these changes Dec 4, 2023

View reviewed changes

mew-ton merged commit aff83a1 into main Dec 4, 2023
1 check passed

mew-ton deleted the fix/75_twist_including_number_key branch December 4, 2023 06:51

github-actions bot mentioned this pull request Dec 4, 2023

Version Packages #83

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: #75 twist 関数で文字列の前方一致していたところを、キーの前方一致判定に変更 #76

fix: #75 twist 関数で文字列の前方一致していたところを、キーの前方一致判定に変更 #76

mew-ton commented Dec 1, 2023 •

edited by coderabbitai bot

Loading

changeset-bot bot commented Dec 1, 2023 •

edited

Loading

coderabbitai bot commented Dec 1, 2023 •

edited

Loading

Chat with CodeRabbit Bot (`@coderabbitai`)

CodeRabbit Commands (invoked as PR comments)

CodeRabbit Configration File (`.coderabbit.yaml`)

mew-ton Dec 1, 2023

coderabbitai bot left a comment

coderabbitai bot Dec 1, 2023 •

edited

Loading

mew-ton Dec 1, 2023

coderabbitai bot Dec 1, 2023

coderabbitai bot left a comment

skadota left a comment

fix: #75 twist 関数で文字列の前方一致していたところを、キーの前方一致判定に変更 #76

fix: #75 twist 関数で文字列の前方一致していたところを、キーの前方一致判定に変更 #76

Conversation

mew-ton commented Dec 1, 2023 • edited by coderabbitai bot Loading

Summary by CodeRabbit

changeset-bot bot commented Dec 1, 2023 • edited Loading

🦋 Changeset detected

coderabbitai bot commented Dec 1, 2023 • edited Loading

Walkthrough

Changes

Chat with CodeRabbit Bot (@coderabbitai)

CodeRabbit Commands (invoked as PR comments)

CodeRabbit Configration File (.coderabbit.yaml)

mew-ton Dec 1, 2023

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Dec 1, 2023 • edited Loading

Choose a reason for hiding this comment

mew-ton Dec 1, 2023

Choose a reason for hiding this comment

coderabbitai bot Dec 1, 2023

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

skadota left a comment

Choose a reason for hiding this comment

mew-ton commented Dec 1, 2023 •

edited by coderabbitai bot

Loading

changeset-bot bot commented Dec 1, 2023 •

edited

Loading

coderabbitai bot commented Dec 1, 2023 •

edited

Loading

Chat with CodeRabbit Bot (`@coderabbitai`)

CodeRabbit Configration File (`.coderabbit.yaml`)

coderabbitai bot Dec 1, 2023 •

edited

Loading