Skip to content

develop#163

Merged
ienaga merged 2 commits intomainfrom
develop
Apr 15, 2026
Merged

develop#163
ienaga merged 2 commits intomainfrom
develop

Conversation

@ienaga
Copy link
Copy Markdown
Member

@ienaga ienaga commented Apr 15, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 15, 2026 04:28
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the configuration documentation (JA/EN/CN) to clarify how stage.json/config.json behave (including warnings about unsupported keys) and bumps the framework package version and several devDependencies.

Changes:

  • Expanded stage.json/config.json docs with clearer semantics (startup load timing, env merge behavior, reserved vs user-defined properties) and added warning callouts.
  • Refined examples/wording for loading and gotoView callbacks, plus section separators.
  • Bumped package.json version to 4.0.3 and updated multiple devDependencies.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
specs/ja/config.md Adds detailed JP explanations + warnings for supported config keys and usage patterns.
specs/en/config.md Adds detailed EN explanations + warnings for supported config keys and usage patterns.
specs/cn/config.md Adds detailed CN explanations + warnings for supported config keys and usage patterns.
package.json Version bump and devDependency updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread specs/ja/config.md
| `width` | number | 240 | 表示領域の幅(ピクセル単位)。実際の描画キャンバスの基準幅として使用される |
| `height` | number | 240 | 表示領域の高さ(ピクセル単位)。実際の描画キャンバスの基準高さとして使用される |
| `fps` | number | 60 | 1秒間の描画回数。指定可能な範囲は 1〜60 |
| `options` | object | null | 追加オプション設定。省略可能 |
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

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

stage.jsonoptions は説明上「省略可能」ですが、型/デフォルトが object / null になっています。フレームワーク側の型定義では stage.options?: IOptionssrc/interface/IStage.ts)なので、ドキュメントも object (optional) もしくは object | undefined(デフォルトは省略/—)に合わせた方が矛盾がありません。

Suggested change
| `options` | object | null | 追加オプション設定。省略可能 |
| `options` | object (optional) | | 追加オプション設定。省略可能 |

Copilot uses AI. Check for mistakes.
Comment thread specs/ja/config.md
Comment on lines +46 to +48
| `fullScreen` | boolean | false | `true` にするとウィンドウ全体にStageを拡張して描画する。`false` の場合は `width`・`height` で指定したサイズで固定描画 |
| `tagId` | string \| null | null | 描画先とするHTML要素のID。指定したIDのエレメント内部にキャンバスが生成される。`null` の場合は `<body>` 直下に生成される |
| `bgColor` | string | "transparent" | 背景色を16進数カラーコードで指定(例: `"#1461A0"`)。`"transparent"` を指定すると透明になる |
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

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

options.tagId の型が string | null、デフォルトが null になっていますが、フレームワークの型定義 IOptionstagId?: stringsrc/interface/IOptions.ts)です。null を許容する設計なら型定義側も含めて揃えるか、ドキュメント側を「省略可(未指定の場合は <body> 直下)」に寄せてください。

Copilot uses AI. Check for mistakes.
Comment thread specs/en/config.md
| `width` | number | 240 | Display area width in pixels. Used as the base width of the rendering canvas |
| `height` | number | 240 | Display area height in pixels. Used as the base height of the rendering canvas |
| `fps` | number | 60 | Number of renders per second. Valid range is 1–60 |
| `options` | object | null | Additional option settings. Can be omitted |
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

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

For stage.json, options is described as “can be omitted” but the table shows type/default as object / null. In the framework typings it is stage.options?: IOptions (src/interface/IStage.ts), so documenting it as optional (or object | undefined with default “—”) would avoid a mismatch.

Suggested change
| `options` | object | null | Additional option settings. Can be omitted |
| `options` | object \| undefined | | Additional option settings. Can be omitted |

Copilot uses AI. Check for mistakes.
Comment thread specs/en/config.md
Comment on lines +46 to +48
| `fullScreen` | boolean | false | When `true`, the Stage expands to fill the entire window. When `false`, rendering is fixed to the size specified by `width` and `height` |
| `tagId` | string \| null | null | ID of the HTML element to use as the rendering target. The canvas is created inside the element with the specified ID. When `null`, the canvas is created directly under `<body>` |
| `bgColor` | string | "transparent" | Background color as a hexadecimal color code (e.g. `"#1461A0"`). Use `"transparent"` for a transparent background |
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

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

options.tagId is documented as string | null with default null, but the framework’s IOptions type is tagId?: string (src/interface/IOptions.ts). Either update the docs to describe it as optional (not null), or update the type definitions if null is truly supported/expected.

Copilot uses AI. Check for mistakes.
Comment thread specs/cn/config.md
| `width` | number | 240 | 显示区域宽度(像素)。作为渲染画布的基准宽度使用 |
| `height` | number | 240 | 显示区域高度(像素)。作为渲染画布的基准高度使用 |
| `fps` | number | 60 | 每秒渲染次数。有效范围为 1〜60 |
| `options` | object | null | 附加选项设置。可省略 |
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

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

stage.jsonoptions 被描述为“可省略”,但表格里类型/默认值写成了 object / null。框架的类型定义是 stage.options?: IOptionssrc/interface/IStage.ts),建议文档改为可选(或 object | undefined,默认值用 “—/省略”),避免与实际类型不一致。

Suggested change
| `options` | object | null | 附加选项设置。可省略 |
| `options` | object(可选) | | 附加选项设置。可省略 |

Copilot uses AI. Check for mistakes.
Comment thread specs/cn/config.md
Comment on lines +46 to +48
| `fullScreen` | boolean | false | 设为 `true` 时,Stage 扩展为填充整个窗口。设为 `false` 时,以 `width`・`height` 指定的尺寸固定渲染 |
| `tagId` | string \| null | null | 用作渲染目标的 HTML 元素 ID。画布在指定 ID 的元素内部创建。为 `null` 时直接在 `<body>` 下创建 |
| `bgColor` | string | "transparent" | 以十六进制颜色代码指定背景色(例:`"#1461A0"`)。指定 `"transparent"` 则为透明 |
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

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

options.tagId 文档写为 string | null 且默认 null,但框架类型 IOptions 中是 tagId?: stringsrc/interface/IOptions.ts)。如果确实允许 null,建议同步更新类型定义;否则文档应改为“可省略/未指定时…”以与类型一致。

Copilot uses AI. Check for mistakes.
@ienaga ienaga merged commit 3009793 into main Apr 15, 2026
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants