Skip to content
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

本番リリース #201

Merged
merged 8 commits into from
Oct 24, 2024
Merged

本番リリース #201

merged 8 commits into from
Oct 24, 2024

Conversation

tegnike
Copy link
Owner

@tegnike tegnike commented Oct 24, 2024

Summary by CodeRabbit

  • 新機能

    • プロジェクトがデュアルライセンスモデル(MITライセンスと商業ライセンス)を採用。
    • 新しい「ライセンス」セクションをREADMEに追加し、商業利用の条件を明確化。
    • 「貢献者のヒント」セクションを追加し、新しい言語の追加方法を説明。
  • ドキュメント

    • FAQセクションを追加し、ライセンス変更に関する詳細を提供。
    • 新しいライセンス文書を追加し、商業ライセンスの価格と範囲を明確化。
  • スタイル

    • READMEおよびライセンス文書のフォーマットを改善し、可読性を向上。

Copy link

vercel bot commented Oct 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aituber-kit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 24, 2024 11:27am

Copy link

coderabbitai bot commented Oct 24, 2024

Walkthrough

LICENSEファイルが単一のMITライセンスから二重ライセンス構造に変更され、非商業利用向けのMITライセンスと商業利用向けの商業ライセンスが明確に区別されました。README.mdでは、プロジェクトの二重ライセンスモデルを告知する新しいセクションが追加され、商業利用に関する情報が整理されています。また、docs/license-faq.mddocs/license.mdなどのドキュメントが新たに追加され、ライセンスに関する詳細な情報やFAQが提供されています。

Changes

ファイル 変更概要
LICENSE MITライセンスから二重ライセンスに変更。非商業利用向けのMITライセンスと商業ライセンスの詳細を追加。商業ライセンスの問い合わせ先も明記。
README.md 二重ライセンスモデルの導入を告知する新セクションを追加。ライセンス情報を再構成し、貢献者向けのヒントセクションを追加。
docs/license-faq.md ライセンス変更に関するFAQを追加。商業ライセンスの必要性や種類、価格、サポートについての情報を詳細に記載。
docs/license-faq_en.md ライセンス変更の理由や新しいライセンスの適用条件を説明するFAQを追加。商業ライセンスの詳細を記載。
docs/license.md 二重ライセンスモデルに関する詳細を記載した新しいドキュメントを追加。MITライセンスと商業ライセンスの条件を明確化。
docs/license_en.md 二重ライセンスモデルに関する詳細な説明を含む新しいドキュメントを追加。商業ライセンスの各種について詳述。
docs/README_en.md 二重ライセンスの導入を告知する新しいセクションを追加。ライセンス情報を明確化し、貢献者向けのヒントを更新。
docs/README_ko.md 二重ライセンスモデルに関する情報を追加。ライセンス情報を更新し、商業利用に関する注意を明記。
docs/README_zh.md 二重ライセンスの導入を告知する新しいセクションを追加。ライセンス情報を更新し、商業利用に関する情報を明確化。

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant Project
    participant License

    User->>Project: 利用する
    alt 非商業利用
        Project->>License: MITライセンス適用
    else 商業利用
        Project->>License: 商業ライセンス必要
        License->>User: 商業ライセンス情報提供
        User->>License: 商業ライセンス購入
    end
Loading

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • 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/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 8

🧹 Outside diff range and nitpick comments (15)
LICENSE (1)

32-33: 非商用ライセンスの注釈の配置について

非商用利用に関する注釈は、ライセンス本文の直後に配置されており適切です。ただし、より目立つように強調することを検討してください。

-*Note: This MIT License applies only to non-commercial use. For commercial use,
-please refer to the Commercial License below.*
+**重要: このMITライセンスは非商用利用のみに適用されます。商用利用の場合は、
+以下の商用ライセンスを参照してください。**
docs/license.md (3)

64-64: メールアドレスのフォーマットを改善してください

セキュリティとアクセシビリティの観点から、メールアドレスをMarkdownリンクとして適切にフォーマットすることを推奨します。

以下の変更を適用してください:

-support@aituberkit.com
+[support@aituberkit.com](mailto:support@aituberkit.com)

Also applies to: 71-71

🧰 Tools
🪛 Markdownlint

64-64: null
Bare URL used

(MD034, no-bare-urls)


31-31: 国際的なユーザーのために価格表示を改善してください

海外のユーザーのために、日本円に加えてUSDでの価格表示も追加することを推奨します。

例えば以下のような形式を検討してください:

-**価格**: 税込み **¥100,000**
+**価格**: 税込み **¥100,000** (約 $700 USD)

Also applies to: 39-39, 47-47


1-2: 目次の追加を推奨します

長文のドキュメントの可読性を向上させるため、目次の追加を推奨します。

ドキュメントの冒頭に以下のような目次を追加することを検討してください:

## 目次

1. [オープンソースライセンス (MIT)](#1-オープンソースライセンス-mit)
2. [商用ライセンス](#2-商用ライセンス)
3. [ライセンスの選択](#ライセンスの選択)
4. [プロジェクトのフォークについて](#プロジェクトのフォークについて)
5. [追加情報](#追加情報)
6. [商用ライセンスの適用範囲](#商用ライセンスの適用範囲)
docs/license-faq.md (3)

93-95: メールアドレスの表記方法を改善してください

セキュリティとスパム対策のため、メールアドレスの表記方法を以下のように変更することを推奨します:

-A14: support@aituberkit.com までお問い合わせください。できる限り迅速に対応いたします。
+A14: [サポート](mailto:support@aituberkit.com)までお問い合わせください。できる限り迅速に対応いたします。
🧰 Tools
🪛 Markdownlint

95-95: null
Bare URL used

(MD034, no-bare-urls)


31-54: 商用ライセンスの価格表示を改善してください

価格表示の一貫性と明確性を高めるため、以下の改善を提案します:

  1. 各ライセンスの価格を同じフォーマットで表示
  2. 税率を明確に記載
  3. 価格改定の可能性について注記を追加
 1. **スタンダードライセンス**
-   - 価格:**税込み ¥100,000**
+   - 価格:¥100,000(税込10%)
    - 利用範囲:
      - 自社内での利用(拠点数やユーザー数無制限)
      - ソフトウェアの改変が可能
      - **再配布・再販は1社まで可能**

 2. **デベロッパーライセンス**
-   - 価格:**税込み ¥300,000**
+   - 価格:¥300,000(税込10%)
    - 利用範囲:
      - ソフトウェアの改変および**再配布が可能(最大5社まで)**
      - 再配布先での拠点数やユーザー数は無制限

 3. **エンタープライズライセンス**
-   - 価格:**税込み ¥1,000,000**
+   - 価格:¥1,000,000(税込10%)
    - 利用範囲:
      - ソフトウェアの改変および**無制限の再配布が可能**
      - 再配布先での拠点数やユーザー数は無制限

 4. **カスタムライセンス**
-   - 価格:**個別見積もり**
+   - 価格:個別見積もり(税別)
    - 利用範囲:特殊な要件や追加の契約条件が必要な場合

+※ 価格は予告なく変更される場合があります。

1-163: 全体的な構成について

ドキュメントの構成は良く整理されていますが、以下の点について追加を検討してください:

  1. 目次の追加
  2. ライセンス違反時の対応についてのセクション
  3. ライセンスの発行手順と支払い方法
  4. 商用ライセンスの譲渡・移転に関する規定

これらの情報を追加することで、ユーザーの理解がより深まり、ライセンス管理がスムーズになると考えられます。必要であれば、これらのセクションの具体的な内容について提案させていただきます。

🧰 Tools
🪛 Markdownlint

95-95: null
Bare URL used

(MD034, no-bare-urls)

docs/license_en.md (2)

57-63: 重要な注意事項の補足が必要です

以下の重要な項目の追加を検討してください:

  1. バージョンアップグレード時の料金体系の詳細
  2. 商用利用開始時期の判断基準
  3. サポート契約のオプションと料金

また、展示会やイベントでのデモ使用に関する免除条件をより具体的に定義することを推奨します。


83-83: 商用ライセンスの適用範囲の明確化が必要です

メインブランチのコードのみが商用利用可能という制限について、以下の点を明確にすることを推奨します:

  1. リリースバージョンの定義
  2. ホットフィックスやパッチの扱い
  3. フォークされたリポジトリでの適用範囲

これらの情報を追加することで、ユーザーの混乱を防ぐことができます。

docs/license-faq_en.md (5)

5-15: ライセンス変更の説明について改善の提案

ライセンス変更の理由と影響について、以下の点を追加することを推奨します:

  • 具体的な改善計画や収益の使用方法
  • バージョン管理の詳細な説明(例:v1.x.xの扱い)
  • 「非営利目的」の明確な定義

21-27: 商用ライセンスが必要なケースの説明を具体化

現在の説明は抽象的です。以下のような具体例を追加することを推奨します:

  • 売上高や従業員数などの具体的な基準
  • 「第三者への提供」の具体的な定義
  • グレーゾーンの事例と判断基準

32-53: 価格体系の説明について改善の提案

価格設定について、以下の情報を追加することを推奨します:

  • 支払い方法や通貨オプション
  • 価格改定の可能性に関する説明
  • カスタムライセンスの具体的な相談プロセス

103-122: 商用ライセンスの適用例をより詳細に

商用ライセンスの例について、以下の点を追加することを推奨します:

  • 各例での具体的な利用規模や条件
  • ライセンス選択の判断フロー
  • 複数のライセンスが適用可能な場合の選択基準

161-163: メインブランチの定義について明確化が必要

メインブランチに関する説明について、以下の点を追加することを推奨します:

  • リリースブランチの扱い
  • バージョンタグの関係
  • 開発ブランチの扱い
README.md (1)

236-246: ライセンスセクションの内容は明確ですが、商用ライセンスについての補足が推奨されます

ライセンスの区分けは明確に説明されていますが、以下の点について追記を検討してください:

  • 商用利用の具体的な定義
  • 商用ライセンスの取得方法
  • 商用ライセンスの価格帯の概要
📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between c91b025 and 93e3926.

📒 Files selected for processing (6)
  • LICENSE (1 hunks)
  • README.md (2 hunks)
  • docs/license-faq.md (1 hunks)
  • docs/license-faq_en.md (1 hunks)
  • docs/license.md (1 hunks)
  • docs/license_en.md (1 hunks)
🧰 Additional context used
🪛 Markdownlint
docs/license-faq.md

95-95: null
Bare URL used

(MD034, no-bare-urls)

docs/license-faq_en.md

95-95: null
Bare URL used

(MD034, no-bare-urls)

docs/license.md

64-64: null
Bare URL used

(MD034, no-bare-urls)


71-71: null
Bare URL used

(MD034, no-bare-urls)

docs/license_en.md

64-64: null
Bare URL used

(MD034, no-bare-urls)


71-71: null
Bare URL used

(MD034, no-bare-urls)

🪛 LanguageTool
docs/license_en.md

[uncategorized] ~5-~5: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...del from version v2.0.0 onwards. ## 1. Open Source License (MIT) The MIT License applies ...

(EN_COMPOUND_ADJECTIVE_INTERNAL)

🔇 Additional comments (9)
LICENSE (2)

18-19: 非商用目的の制限に関する文言の明確化が必要です

MIT ライセンスの標準的な文言に「非商用目的に限る」という制限を追加していますが、この変更がライセンスの有効性に影響を与える可能性があります。法務専門家に確認することを推奨します。


41-41: サポートメールアドレスの確認が必要です

support@aituberkit.com というメールアドレスが使用されていますが、このドメインがプロジェクトの正式なものか確認が必要です。

docs/license.md (2)

3-3: バージョン情報の正確性を確認してください

デュアルライセンス方式の開始バージョンとして記載されている v2.0.0 が、実際のリリースバージョンと一致していることを確認する必要があります。

#!/bin/bash
# バージョン情報の確認
git tag | grep "v2.0.0" || echo "警告: v2.0.0タグが見つかりません"

75-75: FAQドキュメントへのリンクを確認してください

参照されているFAQドキュメント(license-faq.md)が実際に存在することを確認する必要があります。

Also applies to: 79-79

✅ Verification successful

Let me verify the existence of the FAQ document with a different approach.


FAQドキュメントのリンクは正しく設定されています

license-faq.mdファイルはdocsディレクトリに存在することを確認しました。また、英語版のlicense-faq_en.mdも存在しています。リンクの参照先は適切に設定されており、問題ありません。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# FAQドキュメントの存在確認
if [ -f "docs/license-faq.md" ]; then
    echo "FAQドキュメントが存在します"
else
    echo "警告: FAQドキュメントが見つかりません"
fi

Length of output: 168


Script:

#!/bin/bash
# Search for license-faq.md in the repository
fd "license-faq.md"

# Also check if there are any similar FAQ files
fd "faq" --extension md

Length of output: 106

docs/license_en.md (2)

26-31: ライセンス条項の明確化が必要です

Standard LicenseとDeveloper Licenseにおける再配布の制限について、以下の点を明確にすることを推奨します:

  1. 「1社のみ」「最大5社まで」の定義
  2. 再配布先企業の変更可否
  3. 再配布先企業のグループ会社の扱い

これらの条件が不明確な場合、ライセンス適用の解釈に混乱が生じる可能性があります。

Also applies to: 35-39


12-12: ⚠️ Potential issue

著作権表示の更新が必要です

著作権表示が「Nike」となっていますが、これは誤りである可能性が高いです。正しい著作権者名に更新する必要があります。

-Copyright (c) 2024 Nike
+Copyright (c) 2024 [正しい著作権者名]

Likely invalid or redundant comment.

docs/license-faq_en.md (1)

133-146: フォークプロジェクトのライセンスについて明確化が必要

フォークプロジェクトのライセンスについて、以下の点を明確にすることを推奨します:

  • オープンソースライセンスとの互換性
  • コントリビューションの扱い
  • 派生プロジェクトの商用利用条件
✅ Verification successful

Let me gather more information about the license terms and contribution guidelines to provide a comprehensive verification.


収集した情報から、フォークプロジェクトのライセンスについて明確な記述が確認できました。最終的な応答を生成します。

フォークプロジェクトのライセンスは適切に定義されています

ライセンスFAQのQ16およびライセンスドキュメントで、以下の点が明確に定義されています:

  • オープンソースライセンス(MIT)部分のフォークは、商用利用時に商用ライセンスが必要
  • 商用ライセンス部分のフォークは、元のプロジェクトの商用ライセンス条項が適用
  • コントリビューションと派生プロジェクトの商用利用条件は、再配布数に応じて適切な商用ライセンス(スタンダード、デベロッパー、エンタープライズ)の取得が必要
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# フォーク関連の記述を確認
rg -i "fork|derivative|contribute" docs/

Length of output: 1913


Script:

#!/bin/bash
# ライセンスとコントリビューションに関する詳細情報を確認
echo "=== Checking LICENSE files ==="
fd -e md -e txt -i "license|copying" -x cat {}

echo "=== Checking CONTRIBUTING files ==="
fd -i "contributing|contribute" -x cat {}

echo "=== Checking specific sections in docs ==="
rg -i "commercial|opensource|open source|contribution|derivative" docs/license*.md

Length of output: 24562

README.md (2)

5-6: デュアルライセンスの告知が適切に配置されています!

重要な変更点がファイルの先頭に目立つように配置され、ライセンスセクションへのリンクも適切に含まれています。


11-11: ライセンスバッジが新しいライセンスモデルを正確に反映しています!

バッジの表示が「MIT/Commercial」に更新され、デュアルライセンス方式を適切に表現しています。

MIT License (for non-commercial use)
===============================================================================

Copyright (c) 2024 Nike
Copy link

@coderabbitai coderabbitai bot Oct 24, 2024

Choose a reason for hiding this comment

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

⚠️ Potential issue

著作権者名の確認が必要です

著作権者が「Nike」となっていますが、これは誤りである可能性が高いです。正しい組織名に更新する必要があります。

-Copyright (c) 2024 Nike
+Copyright (c) 2024 [正しい組織名]

Committable suggestion was skipped due to low confidence.

Copy link
Owner Author

Choose a reason for hiding this comment

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

@coderabbitai
ニケが私の名前なので問題ないかと、組織もありませんし

Copy link

Choose a reason for hiding this comment

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

@tegnike

ご説明ありがとうございます。著作権者名がご自身のお名前「ニケ」であり、組織は存在しないとのこと、承知いたしました。

(*・ω・)?


✏️ Learnings added
Learnt from: tegnike
PR: tegnike/aituber-kit#201
File: LICENSE:12-12
Timestamp: 2024-10-24T11:09:38.056Z
Learning: 著作権者名は「Nike」であり、組織は存在しない。

Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.

Please contact support@aituberkit.com for details about commercial licensing.

For detailed licensing information, terms and conditions, and FAQ, please visit:
https://github.com/tegnike/aituber-kit/docs/license_en.md
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

ドキュメントURLの修正が必要です

ライセンス情報へのリンクが正しくありません。GitHub上のドキュメントへの正しいパスに修正する必要があります。

-https://github.com/tegnike/aituber-kit/docs/license_en.md
+https://github.com/tegnike/aituber-kit/blob/main/docs/license_en.md
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
https://github.com/tegnike/aituber-kit/docs/license_en.md
https://github.com/tegnike/aituber-kit/blob/main/docs/license_en.md

Comment on lines +57 to +62
- すべての商用ライセンスは**買い切り制**です。
- ライセンスに**期限はありません**(永続的に利用可能)。
- ライセンスには**サポートは含まれません**。サポートが必要な場合は、別途ご相談ください。
- ライセンス取得時の**最新バージョン**に適用されます。メジャーアップデートがあった場合、アップグレード料金が発生する可能性があります。
- 営利目的以外での個人利用、教育目的、非営利目的の使用として使用を開始した後、**商用利用が確認された場合**には、商用利用開始時からの商用ライセンス料金を請求させていただきます。
- プロジェクトの認知度向上を目的とした**展示会やイベントでのデモンストレーション**は、商用ライセンスの対象外となります。
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

商用利用の定義を明確にすることを推奨します

「商用利用」の具体的な定義や判断基準を追加することで、ユーザーがより適切にライセンスを選択できるようになります。

以下のような内容の追加を検討してください:

+## 商用利用の定義
+
+以下のような使用は「商用利用」とみなされます:
+- 営利企業による社内での使用
+- 収益を生み出す製品やサービスでの使用
+- 有料のサービスや製品の一部としての使用
+- クライアントワークでの使用
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- すべての商用ライセンスは**買い切り制**です。
- ライセンスに**期限はありません**(永続的に利用可能)。
- ライセンスには**サポートは含まれません**。サポートが必要な場合は、別途ご相談ください。
- ライセンス取得時の**最新バージョン**に適用されます。メジャーアップデートがあった場合、アップグレード料金が発生する可能性があります。
- 営利目的以外での個人利用、教育目的、非営利目的の使用として使用を開始した後、**商用利用が確認された場合**には、商用利用開始時からの商用ライセンス料金を請求させていただきます。
- プロジェクトの認知度向上を目的とした**展示会やイベントでのデモンストレーション**は、商用ライセンスの対象外となります。
- すべての商用ライセンスは**買い切り制**です。
- ライセンスに**期限はありません**(永続的に利用可能)。
- ライセンスには**サポートは含まれません**。サポートが必要な場合は、別途ご相談ください。
- ライセンス取得時の**最新バージョン**に適用されます。メジャーアップデートがあった場合、アップグレード料金が発生する可能性があります。
- 営利目的以外での個人利用、教育目的、非営利目的の使用として使用を開始した後、**商用利用が確認された場合**には、商用利用開始時からの商用ライセンス料金を請求させていただきます。
- プロジェクトの認知度向上を目的とした**展示会やイベントでのデモンストレーション**は、商用ライセンスの対象外となります。
## 商用利用の定義
以下のような使用は「商用利用」とみなされます:
- 営利企業による社内での使用
- 収益を生み出す製品やサービスでの使用
- 有料のサービスや製品の一部としての使用
- クライアントワークでの使用

Comment on lines +67 to +68
A8: 一般的に、個人的な用途での利用は商用利用には当たらず、MITライセンスでの利用が可能です。ただし、大規模な収益が発生する場合は商用ライセンスの購入をご検討ください。

Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

収益基準の明確化が必要です

「大規模な収益」の定義が曖昧で、ユーザーが判断に迷う可能性があります。

-A8: 一般的に、個人的な用途での利用は商用利用には当たらず、MITライセンスでの利用が可能です。ただし、大規模な収益が発生する場合は商用ライセンスの購入をご検討ください。
+A8: 一般的に、個人的な用途での利用は商用利用には当たらず、MITライセンスでの利用が可能です。ただし、月間収益が50万円を超える場合は商用ライセンスの購入が必要です。
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
A8: 一般的に、個人的な用途での利用は商用利用には当たらず、MITライセンスでの利用が可能です。ただし、大規模な収益が発生する場合は商用ライセンスの購入をご検討ください
A8: 一般的に、個人的な用途での利用は商用利用には当たらず、MITライセンスでの利用が可能です。ただし、月間収益が50万円を超える場合は商用ライセンスの購入が必要です

Comment on lines +159 to +160
A19: プロジェクトの認知度向上を目的とした展示会やイベントでのデモンストレーションは、**MITライセンスで利用可能**です。これは、商用イベントであっても適用されます。ただし、展示会やイベントでの利用が直接的な収益につながる場合は、商用ライセンスが必要となる可能性があります。

Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

イベント利用に関する基準の明確化が必要です

「直接的な収益」の定義が曖昧で、解釈に幅が生じる可能性があります。

-A19: プロジェクトの認知度向上を目的とした展示会やイベントでのデモンストレーションは、**MITライセンスで利用可能**です。これは、商用イベントであっても適用されます。ただし、展示会やイベントでの利用が直接的な収益につながる場合は、商用ライセンスが必要となる可能性があります。
+A19: プロジェクトの認知度向上を目的とした展示会やイベントでのデモンストレーションは、**MITライセンスで利用可能**です。これは、商用イベントであっても適用されます。ただし、以下の場合は商用ライセンスが必要です:
+
+- イベントでの製品・サービスの販売
+- 有料のワークショップやセミナーでの使用
+- 展示会での商用デモ(商談目的)
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
A19: プロジェクトの認知度向上を目的とした展示会やイベントでのデモンストレーションは、**MITライセンスで利用可能**です。これは、商用イベントであっても適用されます。ただし、展示会やイベントでの利用が直接的な収益につながる場合は、商用ライセンスが必要となる可能性があります。
A19: プロジェクトの認知度向上を目的とした展示会やイベントでのデモンストレーションは、**MITライセンスで利用可能**です。これは、商用イベントであっても適用されます。ただし、以下の場合は商用ライセンスが必要です:
- イベントでの製品・サービスの販売
- 有料のワークショップやセミナーでの使用
- 展示会での商用デモ(商談目的)

- If commercial use is confirmed after beginning use for non-profit personal, educational, or non-profit purposes, commercial license fees will be charged from the start of commercial use.
- **Demonstrations at exhibitions and events** aimed at increasing project awareness are exempt from commercial licensing requirements.

For commercial license details and purchase information, please contact support@aituberkit.com.
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

メールアドレスの記載方法の改善が必要です

セキュリティとスパム対策の観点から、メールアドレスの記載方法を以下のように変更することを推奨します:

-support@aituberkit.com
+`support [at] aituberkit [dot] com`

Also applies to: 71-71

🧰 Tools
🪛 Markdownlint

64-64: null
Bare URL used

(MD034, no-bare-urls)


## Regarding Project Forks

When **forking this project**, these license terms apply to the forked project. Commercial use or redistribution of forks may require separate commercial licenses. For details, please refer to the [Project FAQ](license-faq_en.md).
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

リンクの修正が必要です

Markdownのリンク記法が相対パスを使用していますが、絶対パスを使用することを推奨します。また、リンク切れを防ぐため、パスの正確性を確認してください。

-[Project FAQ](license-faq_en.md)
+[Project FAQ](/docs/license-faq_en.md)
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
When **forking this project**, these license terms apply to the forked project. Commercial use or redistribution of forks may require separate commercial licenses. For details, please refer to the [Project FAQ](license-faq_en.md).
When **forking this project**, these license terms apply to the forked project. Commercial use or redistribution of forks may require separate commercial licenses. For details, please refer to the [Project FAQ](/docs/license-faq_en.md).


**Q14: How can I ask additional questions about licensing?**

A14: Please contact support@aituberkit.com. We will respond as promptly as possible.
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

メールアドレスの表記方法の修正が必要

生のURLやメールアドレスは、スパム対策のリスクがあります。以下のような方法での記載を推奨します:

  • Markdownのリンク形式を使用
  • 「お問い合わせフォーム」へのリンクを追加

以下の修正を提案します:

-A14: Please contact support@aituberkit.com. We will respond as promptly as possible.
+A14: Please contact [our support team](mailto:support@aituberkit.com). We will respond as promptly as possible.
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
A14: Please contact support@aituberkit.com. We will respond as promptly as possible.
A14: Please contact [our support team](mailto:support@aituberkit.com). We will respond as promptly as possible.
🧰 Tools
🪛 Markdownlint

95-95: null
Bare URL used

(MD034, no-bare-urls)

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 3

🧹 Outside diff range and nitpick comments (1)
docs/README_en.md (1)

239-249: 商用ライセンスの問い合わせ方法を追加することを推奨します

ライセンスセクションは明確に構造化されていますが、商用ライセンスに関する具体的な問い合わせ先情報が不足しています。以下の追加を提案します:

 - **Commercial License**
   - A separate commercial license is required for commercial use.
   - For details, please check [About License](./license_en.md).
+  - For commercial license inquiries, please contact us at [contact information].
🧰 Tools
🪛 LanguageTool

[uncategorized] ~243-~243: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...MIT License + Commercial License). - Open Source License (MIT) - MIT License applies...

(EN_COMPOUND_ADJECTIVE_INTERNAL)

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between 93e3926 and 9778f04.

📒 Files selected for processing (3)
  • docs/README_en.md (3 hunks)
  • docs/README_ko.md (3 hunks)
  • docs/README_zh.md (2 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/README_en.md

[uncategorized] ~243-~243: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...MIT License + Commercial License). - Open Source License (MIT) - MIT License applies...

(EN_COMPOUND_ADJECTIVE_INTERNAL)

docs/README_zh.md

[uncategorized] ~245-~245: “关于”组成的介词短语必需位于句首,或请改用"对于"代替。
Context: ...证** - 商业目的的使用需要另外获取商业许可证。 - 详细信息请查看关于许可证。 ### 其他 - [标志使用协...

(wb2)

🪛 Markdownlint
docs/README_en.md

329-329: null
Fenced code blocks should have a language specified

(MD040, fenced-code-language)

🔇 Additional comments (9)
docs/README_zh.md (3)

5-5: 通知バナーの配置と内容が適切です

通知バナーは目立つ位置に配置され、デュアルライセンスの重要な変更を明確に伝えています。商用利用に関する詳細へのリンクも適切に設定されています。


11-11: ライセンスバッジの更新が正確です

MIT/Commercialの新しいデュアルライセンス構造を正確に反映したバッジに更新されています。


249-250: 英語版ドキュメントへのリンク更新が適切です

商標使用規約とVRMモデル使用規約への英語版リンクが正しく更新されています。

docs/README_ko.md (4)

5-6: 通知セクションの配置と内容が適切です

デュアルライセンスの採用について、ドキュメントの最上部に目立つ形で配置されており、ユーザーへの周知が効果的に行われています。


11-11: ライセンスバッジの更新が正確です

GitHubバッジが新しいデュアルライセンスモデル(MIT/Commercial)を正確に反映しています。


237-247: ライセンスセクションの構成が明確です

ライセンスの説明が以下の点で適切に構成されています:

  • デュアルライセンスの採用が明確に説明されています
  • 各ライセンスの適用範囲が具体的に示されています
  • 商用ライセンスの詳細情報への参照が適切に提供されています

248-251: 関連ドキュメントへのリンクが適切です

ロゴとVRMモデルの利用規約へのリンクが明確に提供されており、ユーザーが必要な情報に容易にアクセスできます。

docs/README_en.md (2)

5-6: ライセンス変更の通知が適切に追加されています!

バージョンv2.0.0からのデュアルライセンスアプローチへの移行が明確に通知されており、商用利用に関する参照先も適切に示されています。


11-11: バッジの更新が適切に行われています!

ライセンスバッジが「MIT/Commercial」に更新され、新しいライセンスモデルを正確に反映しています。

Comment on lines +234 to +246
## 使用协议

### 许可证

从版本v2.0.0开始,本项目采用**双重许可方式**(MIT许可证 + 商业许可证)。

- **开源许可证 (MIT)**
- MIT许可证适用于非营利目的的个人使用、教育目的和非营利目的的使用。

- **商业许可证**
- 商业目的的使用需要另外获取商业许可证。
- 详细信息请查看[关于许可证](./license_en.md)。

Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

ライセンスセクションの構造と内容について改善の提案

ライセンスセクションは明確に構造化されていますが、以下の改善を提案します:

  1. 商用ライセンスの具体的な取得方法や問い合わせ先の情報を追加
  2. 非営利目的の具体例を追加することで、ユーザーの理解を促進

以下のような内容の追加を提案します:

 ### 许可证

 从版本v2.0.0开始,本项目采用**双重许可方式**(MIT许可证 + 商业许可证)。

 - **开源许可证 (MIT)**
   - MIT许可证适用于非营利目的的个人使用、教育目的和非营利目的的使用。
+   - 例如:个人学习、学术研究、非营利组织的使用等

 - **商业许可证**
   - 商业目的的使用需要另外获取商业许可证。
+   - 商业许可证的获取方式:
+     1. 通过GitHub Sponsors赞助项目
+     2. 发送邮件至[联系邮箱]进行咨询
   - 详细信息请查看[关于许可证](./license_en.md)。

Committable suggestion was skipped due to low confidence.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~245-~245: “关于”组成的介词短语必需位于句首,或请改用"对于"代替。
Context: ...证** - 商业目的的使用需要另外获取商业许可证。 - 详细信息请查看关于许可证。 ### 其他 - [标志使用协...

(wb2)

@@ -161,7 +163,7 @@ npm run dev

### 기타

- 대화 내역은 설정 화면에서 재설정할 수 있습니다.
- 대화 내역은 설정 화면에서 재설정할 수 있��니다.
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

文字化けを修正してください

会話履歴のリセットに関する説明文に文字化けが発生しています。

以下の修正を提案します:

-대화 내역은 설정 화면에서 재설정할 수 있��니다.
+대화 내역은 설정 화면에서 재설정할 수 있습니다.
📝 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. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- 대화 내역은 설정 화면에서 재설정할 수 있��니다.
대화 내역은 설정 화면에서 재설정할 수 있습니다.

Comment on lines +329 to +330
```
</rewritten_chunk>
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

空のコードブロックを削除してください

ファイルの最後に不要な空のコードブロックが存在します。このブロックは削除する必要があります。

🧰 Tools
🪛 Markdownlint

329-329: null
Fenced code blocks should have a language specified

(MD040, fenced-code-language)

@tegnike tegnike merged commit e6b9750 into main Oct 24, 2024
3 checks passed
@tegnike tegnike deleted the release-2.0.0 branch October 24, 2024 15:21
@coderabbitai coderabbitai bot mentioned this pull request Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant