Merge branch 'master' into aiscript-next

Author: takejohn

CHANGELOG.md

@@ -1,5 +1,14 @@
 [Read translated version (en)](./translations/en/CHANGELOG.md)
 
+# 1.1.1
+
+- Fix: オブジェクトリテラルのプロパティ名に一部の予約語を記述できなかった問題を修正
+
+# 1.1.0
+
+- オブジェクトリテラルのプロパティ名に予約語を直接記述できるようになりました。
+- Fix: `Math:gen_rng`のアルゴリズム`chacha20`および`rc4`が非セキュアコンテクスト下では動作しないため、そのような環境下では`options.algorithm`のデフォルトを`rc4_legacy`に変更
+
 # 1.0.0
 
 - 新しいAiScriptパーサーを実装

CONTRIBUTING.md

@@ -33,7 +33,7 @@ PRを作る際は、`npm run api`コマンドを実行してAPIレポートを
 ### Codecov
 このプロジェクトではカバレッジの計測に[Codecov](https://about.codecov.io/)を導入しています。カバレッジは、コードがどれくらいテストでカバーされているかを表すものです。
 
-カバレッジ計測はCIで自動的に行われ、特に操作は必要ありません。カバレッジは[ここ](https://codecov.io/gh/syuilo/aiscript)から見ることができます。
+カバレッジ計測はCIで自動的に行われ、特に操作は必要ありません。カバレッジは[ここ](https://codecov.io/gh/aiscript-dev/aiscript)から見ることができます。
 
 また、各PRに対してもそのブランチのカバレッジが自動的に計算され、マージ先のカバレッジとの差分を含んだレポートがCodecovのbotによりコメントされます。これにより、そのPRをマージすることでどれくらいカバレッジが増加するのか/減少するのかを確認することができます。
 

docs/branches-and-releasing.md

@@ -0,0 +1,58 @@
+このドキュメントは、記載時点でのブランチの運用方法を記録したaiscript-dev member向けの参考資料です。  
+現在の実態とは乖離した内容が記載されている可能性があります。  
+
+## ブランチの運用
+### masterブランチ
+デフォルトブランチであり、マイナーリリース用ブランチです。非破壊的変更のプルリクエストはここ向けに作成してください。  
+[publish-nightly](/.github/workflows/publish-nightly.yml)ワークフローにより、変更があると開発中バージョン(`dev-<DATE>`)がpublishされます。  
+### aiscript-nextブランチ
+メジャーリリース用ブランチで、破壊的変更のプルリクエストはここ向けに作成してください。  
+定期的にmasterブランチをマージします。（このときはsquashマージは使用しないでください）  
+[publish-nightly](/.github/workflows/publish-nightly.yml)ワークフローにより、変更があると開発中バージョン(`next-<DATE>`)がpublishされます。  
+メジャーリリースが近いときはmasterに合流されていることもあります。その場合は破壊的変更・非破壊的変更共にmasterにpushします。  
+### gh-pagesブランチ
+[簡易playground](https://aiscript-dev.github.io/aiscript/)のデプロイ用です。  
+### bugfix用ブランチ
+特にブランチ名は決まっていません。bugfix用に作成され、リリースされると削除されます。  
+作成時は前回リリース時のコミットから分岐してください。  
+### その他のブランチ
+特に用途は決まっていません。aiscript-devのmemberが自由に作成・使用することができます。  
+dependabotのPRや古いプロジェクトのアーカイブなどがあります。  
+
+## 新バージョンをリリースするときの流れ
+### bugfixリリース
+バグ修正。できるだけ早急にリリースを目指す。バージョンx.y.zのzを一つ上げる。
+- bugfix用のブランチを作成。前回リリース時のコミットから分岐する。
+  - master（マイナーリリース用ブランチ）ではないので注意。
+- bugfixブランチに修正用PRをマージ/コミットをプッシュ。
+- bugfixブランチで`npm version patch && npm run pre-release`を実行し、結果をプッシュ。
+- bugfixブランチで`npm publish`。
+- bugfixブランチで`git tag <バージョン名> && git push --tag`。
+- bugfixブランチをmasterブランチに合流。
+  - package.jsonなどのバージョンにコンフリクトが生じた場合、masterのものを反映。
+### マイナーリリース
+非破壊的変更。バージョンx.y.zのyを一つ上げ、zを0にする。
+- 事前にdependabotの更新があれば極力マージする。
+- masterブランチで`npm run pre-release`を実行し、結果をプッシュ。
+- masterブランチで`npm publish`。
+- masterブランチで`git tag <バージョン名> && git push --tag`。
+- masterブランチで`npm version minor && npm run build && npm run api`を実行、結果をプッシュ。
+### メジャーリリース
+破壊的変更。バージョンx.y.zのxを一つ上げ、yとzを0にする。
+- aiscript-nextブランチをmasterブランチに合流。
+  - package.jsonなどのバージョンにコンフリクトが生じた場合、aiscript-nextのものを反映。
+  - この合流は、次回リリースがメジャーリリースになると決まった段階でいつでもおこなってよい。
+- dependabotの更新があれば極力マージする。
+- masterブランチで`npm run pre-release`を実行し、結果をプッシュ。
+- masterブランチで`npm publish`。
+- masterブランチで`git tag <バージョン名> && git push --tag`。
+- masterブランチからaiscript-nextブランチを再作成。
+- aiscript-nextブランチで`npm version major && npm run build && npm run api`を実行、結果をプッシュ。
+- masterブランチで`npm version minor && npm run build && npm run api`を実行、結果をプッシュ。
+### 補遺
+#### publish後にnpm version minor/majorをするのはなぜ？
+本リポジトリにおけるpackage.jsonのバージョンは、masterは「次のマイナーリリースのバージョン」、aiscript-nextは「次のメジャーリリースのバージョン」の番号で保持することにしています。  
+これは主に、毎晩自動で行われる開発中バージョンのpublishの都合に関係しています。
+開発中（プレリリース）バージョンのバージョン番号は、[semantic versioning 2.0の定義では](https://semver.org/lang/ja/#spec-item-9)「次回リリース予定のバージョン」にハイフンを挟んで何かしらの文字列を置くということになっています。  
+このバージョニングを機械的に行うには、元々「次回リリース予定のバージョン」になっているpackage.jsonのバージョンに適当な文字列を接尾するという形が都合がいいのです。  
+（ちなみに設備される文字列は、masterのプレリリースでは`dev-<リリース日時>`、aiscript-nextでは`next-<リリース日時>`になっています）

eslint.config.mjs

@@ -1,4 +1,5 @@
 import importPlugin from "eslint-plugin-import";
+import unusedImports from "eslint-plugin-unused-imports";
 import js from "@eslint/js";
 import ts from 'typescript-eslint';
 
@@ -13,6 +14,9 @@ export default ts.config({
 		importPlugin.flatConfigs.recommended,
 		importPlugin.flatConfigs.typescript,
 	],
+	plugins: {
+		"unused-imports": unusedImports,
+	},
 
 	languageOptions: {
 		ecmaVersion: 5,
@@ -116,10 +120,15 @@ export default ts.config({
 			],
 		}],
 
-		"@typescript-eslint/no-unused-vars": ["warn", {
-			argsIgnorePattern: "^_",
+		"@typescript-eslint/no-unused-vars": "off",
+		"unused-imports/no-unused-imports": "warn",
+		"unused-imports/no-unused-vars": ["warn", {
+			vars: "all",
 			varsIgnorePattern: "^_",
-			caughtErrorsIgnorePattern: "^_",
+			args: "after-used",
+			argsIgnorePattern: "^_",
+			caughtErrors: "none",
+			// caughtErrorsIgnorePattern: "^_",
 			destructuredArrayIgnorePattern: "^_",
 		}],
 	},

package-lock.json

@@ -42,6 +42,7 @@
 		"chalk": "5.5.0",
 		"eslint": "9.32.0",
 		"eslint-plugin-import": "2.32.0",
+		"eslint-plugin-unused-imports": "4.2.0",
 		"semver": "7.7.2",
 		"ts-node": "10.9.2",
 		"typescript": "5.9.2",

package.json

@@ -1,4 +1,3 @@
-import { TokenKind } from './parser/token.js';
 import type { Pos } from './node.js';
 
 export abstract class AiScriptError extends Error {

src/error.ts

@@ -265,7 +265,7 @@ export class Interpreter {
 					) {
 						value.name = nsScope.getNsPrefix() + node.dest.name;
 					}
-					await this.define(nsScope, node.dest, value, node.mut);
+					this.define(nsScope, node.dest, value, node.mut);
 
 					break;
 				}
@@ -309,7 +309,7 @@ export class Interpreter {
 					) {
 						value.name = nsScope.getNsPrefix() + node.dest.name;
 					}
-					this.defineSync(nsScope, node.dest, value, node.mut);
+					this.define(nsScope, node.dest, value, node.mut);
 
 					break;
 				}
@@ -668,7 +668,7 @@ export class Interpreter {
 				) {
 					value.name = node.dest.name;
 				}
-				await this.define(scope, node.dest, value, node.mut);
+				this.define(scope, node.dest, value, node.mut);
 				return NULL;
 			}
 
@@ -1212,7 +1212,7 @@ export class Interpreter {
 				) {
 					value.name = node.dest.name;
 				}
-				this.defineSync(scope, node.dest, value, node.mut);
+				this.define(scope, node.dest, value, node.mut);
 				return NULL;
 			}
 
@@ -1667,34 +1667,7 @@ export class Interpreter {
 	}
 
 	@autobind
-	private async define(scope: Scope, dest: Ast.Expression, value: Value, isMutable: boolean): Promise<void> {
-		switch (dest.type) {
-			case 'identifier': {
-				scope.add(dest.name, { isMutable, value });
-				break;
-			}
-			case 'arr': {
-				assertArray(value);
-				await Promise.all(dest.value.map(
-					(item, index) => this.define(scope, item, value.value[index] ?? NULL, isMutable),
-				));
-				break;
-			}
-			case 'obj': {
-				assertObject(value);
-				await Promise.all([...dest.value].map(
-					([key, item]) => this.define(scope, item, value.value.get(key) ?? NULL, isMutable),
-				));
-				break;
-			}
-			default: {
-				throw new AiScriptRuntimeError('The left-hand side of an definition expression must be a variable.');
-			}
-		}
-	}
-
-	@autobind
-	private defineSync(scope: Scope, dest: Ast.Expression, value: Value, isMutable: boolean): void {
+	private define(scope: Scope, dest: Ast.Expression, value: Value, isMutable: boolean): void {
 		switch (dest.type) {
 			case 'identifier': {
 				scope.add(dest.name, { isMutable, value });
@@ -1703,14 +1676,14 @@ export class Interpreter {
 			case 'arr': {
 				assertArray(value);
 				dest.value.map(
-					(item, index) => this.defineSync(scope, item, value.value[index] ?? NULL, isMutable),
+					(item, index) => this.define(scope, item, value.value[index] ?? NULL, isMutable),
 				);
 				break;
 			}
 			case 'obj': {
 				assertObject(value);
 				[...dest.value].map(
-					([key, item]) => this.defineSync(scope, item, value.value.get(key) ?? NULL, isMutable),
+					([key, item]) => this.define(scope, item, value.value.get(key) ?? NULL, isMutable),
 				);
 				break;
 			}