Skip to content

Develop#36

Merged
ienaga merged 2 commits into
mainfrom
develop
Apr 15, 2026
Merged

Develop#36
ienaga merged 2 commits into
mainfrom
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 05:06
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 project’s release/version metadata, refreshes build tool dependencies (Astro/TypeScript and transitive lockfile updates), and expands the framework config documentation across JA/EN/CN.

Changes:

  • Bump package version to 2.0.4 and update astro and typescript devDependencies.
  • Regenerate package-lock.json to reflect updated dependency graph.
  • Improve framework config docs content/structure across Japanese/English/Chinese pages.

Reviewed changes

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

Show a summary per file
File Description
package.json Version bump and devDependency updates (Astro/TypeScript)
package-lock.json Lockfile refresh reflecting new dependency graph and engine constraints
docs/ja/reference/framework/config/index.html Expanded/clarified config documentation (JA)
docs/en/reference/framework/config/index.html Expanded/clarified config documentation (EN)
docs/cn/reference/framework/config/index.html Expanded/clarified config documentation (CN)
Comments suppressed due to low confidence (1)

package.json:22

  • The lockfile indicates astro@6.1.6 requires Node >=22.12.0 (see its engines field). Since package.json doesn’t declare an engines.node, consumers may only find out at install/runtime. Please either (1) add an explicit engines entry (and align CI/docs), or (2) pin Astro to a version compatible with the project’s supported Node range.
  "scripts": {
    "start": "astro dev",
    "build": "astro build",
    "preview": "astro preview",
    "astro": "astro"
  },
  "devDependencies": {
    "typescript": "^6.0.2",
    "@astrojs/sitemap": "^3.7.2",
    "aos": "^2.3.4",
    "astro": "^6.1.6",
    "bootstrap": "^5.3.8",
    "bootstrap-icons": "^1.13.1",
    "unist-util-visit": "^5.1.0"
  },

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

Comment on lines +95 to +101
<blockquote>
<p>[!WARNING]
The only valid properties in <code>stage.json</code> are <code>width</code>, <code>height</code>, <code>fps</code>, and <code>options</code>.
Properties such as <code>align</code> and <code>scaleMode</code> — even though they relate to Stage display — do not exist in <code>stage.json</code>.
If you need these settings, define them in <code>config.json</code>.
Any properties not listed above are completely ignored by the framework.</p>
</blockquote>
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.

This warning callout is written as a literal [!WARNING] marker inside the HTML output. Unless your site CSS/JS explicitly transforms this pattern, it will render as raw text to readers. Consider replacing it with explicit HTML for a warning callout (e.g., a styled <div class="admonition warning"> with a heading) or ensure the build pipeline converts admonition syntax before HTML is emitted.

Suggested change
<blockquote>
<p>[!WARNING]
The only valid properties in <code>stage.json</code> are <code>width</code>, <code>height</code>, <code>fps</code>, and <code>options</code>.
Properties such as <code>align</code> and <code>scaleMode</code> — even though they relate to Stage display — do not exist in <code>stage.json</code>.
If you need these settings, define them in <code>config.json</code>.
Any properties not listed above are completely ignored by the framework.</p>
</blockquote>
<div class="admonition warning">
<p><strong>Warning</strong></p>
<p>The only valid properties in <code>stage.json</code> are <code>width</code>, <code>height</code>, <code>fps</code>, and <code>options</code>.
Properties such as <code>align</code> and <code>scaleMode</code> — even though they relate to Stage display — do not exist in <code>stage.json</code>.
If you need these settings, define them in <code>config.json</code>.
Any properties not listed above are completely ignored by the framework.</p>
</div>

Copilot uses AI. Check for mistakes.
Comment on lines +95 to +100
<blockquote>
<p>[!WARNING]
<code>stage.json</code> で有効なプロパティは上記の <code>width</code>・<code>height</code>・<code>fps</code>・<code>options</code> のみです。
<code>align</code>・<code>scaleMode</code> などステージ表示に関わる設定であっても、<code>stage.json</code> には存在しないプロパティです。
これらの設定が必要な場合は <code>config.json</code> に記述してください。
上記以外のプロパティを記述しても、フレームワークは一切処理しません。</p>
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.

This warning callout is written as a literal [!WARNING] marker inside the HTML output. Unless your site CSS/JS explicitly transforms this pattern, it will render as raw text to readers. Consider replacing it with explicit HTML for a warning callout (e.g., a styled <div class="admonition warning"> with a heading) or ensure the build pipeline converts admonition syntax before HTML is emitted.

Copilot uses AI. Check for mistakes.
Comment on lines +95 to +100
<blockquote>
<p>[!WARNING]
<code>stage.json</code> 中有效的属性仅有 <code>width</code>・<code>height</code>・<code>fps</code>・<code>options</code>。
<code>align</code>・<code>scaleMode</code> 等与 Stage 显示相关的设置,在 <code>stage.json</code> 中并不存在。
如需这些设置,请在 <code>config.json</code> 中定义。
上述以外的属性将被框架完全忽略,不做任何处理。</p>
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.

This warning callout is written as a literal [!WARNING] marker inside the HTML output. Unless your site CSS/JS explicitly transforms this pattern, it will render as raw text to readers. Consider replacing it with explicit HTML for a warning callout (e.g., a styled <div class="admonition warning"> with a heading) or ensure the build pipeline converts admonition syntax before HTML is emitted.

Copilot uses AI. Check for mistakes.
Comment thread package.json
"devDependencies": {
"@astrojs/check": "^0.9.8",
"typescript": "^5.9.3",
"typescript": "^6.0.2",
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.

typescript is bumped to v6, but the lockfile shows some tooling (e.g. tsconfck) declaring a peer range of typescript: ^5.0.0. This will produce peer dependency warnings and may break type tooling if TypeScript 6 introduces incompatibilities. Consider pinning TypeScript to a compatible v5 release (or upgrading the dependent tooling) and verifying astro dev/build works cleanly without peer warnings.

Suggested change
"typescript": "^6.0.2",
"typescript": "^5.9.3",

Copilot uses AI. Check for mistakes.
@ienaga ienaga merged commit 9dd6612 into main Apr 15, 2026
8 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