3 Commits

Author SHA1 Message Date
79d46bdad1 chore: bump version 2025-09-12 02:11:03 +02:00
74a724c287 feat: make path in error message relative to rootDir 2025-09-12 02:08:24 +02:00
6c3889d2a0 chore: add README 2025-09-12 02:05:41 +02:00
6 changed files with 107 additions and 15 deletions

90
README.md Normal file
View File

@@ -0,0 +1,90 @@
<h1>SpyglassMC Action</h1>
This action lets you validate your Minecraft datapack using [SpyglassMC](https://spyglassmc.com/).
All JSON and MCFunction files are checked for syntax errors.
<details>
<summary><strong>Table of Contents</strong></summary>
---
- [Basic workflow](#basic-workflow)
- [Input parameters](#input-parameters)
- [GitHub vs. Gitea](#github-vs-gitea)
- [Gitea](#gitea)
- [GitHub](#github)
---
</details>
## Basic workflow
Here is a simple example showing how to use this action:
```yaml
name: Check datapack
on:
pull_request:
branches:
- main
jobs:
check:
runs-on: ubuntu-latest
steps:
- name: Checkout code
uses: actions/checkout@v4
- name: Check syntax and schemas
uses: https://git.kb28.ch/HEL/spyglassmc-action@v1.1.4
```
## Input parameters
The following parameters can be passed using the `with` section:
|Parameter|Type|Default|Description|
|:---|:---|:---:|:---|
|`rootDir` |Path |`./` |Root datapack directory (containing the data folder and pack.mcmeta file)|
|`version` |String |`auto` |The Minecraft version to use. If not specified or set to 'auto', will try to use the version indicated in `pack.mcmeta`|
|`reportAllErrors`|Boolean|`false`|Whether to report all errors or only the first one|
|`verbose` |Boolean|`false`|Whether to provide details on errors or just the number of errors|
## GitHub vs. Gitea
Depending on where you want to execute this action, you should be mindful of the following points
### Gitea
If your workflow will be running on Gitea, you might need to set a specific runner image (see https://gitea.com/gitea/runner-images#migration). For example:
```yaml
...
jobs:
check:
runs-on: ubuntu-latest
container: catthehacker/ubuntu:act-latest
...
```
### GitHub
If your workflow will be running on GitHub, you will not be able to simply use the action from this Gitea repository.\
Instead, you need to add a step to clone this repository, and use the action locally
```yaml
name: Check datapack
on:
pull_request:
branches:
- main
jobs:
deploy:
runs-on: ubuntu-latest
steps:
- name: Checkout code
uses: actions/checkout@v4
- name: Checkout action
run: |
git clone https://git.kb28.ch/HEL/spyglassmc-action ./.github/actions/spyglassmc-action
- name: Check syntax and schemas
uses: ./.github/actions/spyglassmc-action
```
You might also want to `git checkout` a specific tagged version

View File

@@ -19,7 +19,7 @@ export declare class CustomService extends core.Service {
private getFileContent; private getFileContent;
private getLineAndColumn; private getLineAndColumn;
private getErrorProperties; private getErrorProperties;
checkFile(path: string, lang: string): Promise<boolean>; checkFile(filePath: string, lang: string): Promise<boolean>;
checkAllFiles(): Promise<boolean>; checkAllFiles(): Promise<boolean>;
} }
export declare function checkDatapack(rootDir: string, version: string, reportAll: boolean, verbose: boolean): Promise<void>; export declare function checkDatapack(rootDir: string, version: string, reportAll: boolean, verbose: boolean): Promise<void>;

View File

@@ -1 +1 @@
{"version":3,"file":"","sourceRoot":"","sources":["file:///home/louis/Documents/spyglassmc-action/src/check_datapack.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,IAAI,MAAM,kBAAkB,CAAA;AAQxC,MAAM,WAAW,aAAa;IAC1B;;OAEG;IACH,eAAe,EAAE,OAAO,CAAA;IAExB;;OAEG;IACH,OAAO,EAAE,OAAO,CAAA;CACnB;AAkBD,qBAAa,aAAc,SAAQ,IAAI,CAAC,OAAO;IAC3C,OAAO,EAAE,MAAM,CAAA;IACf,OAAO,EAAE,aAAa,CAAA;gBAEV,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa;IAiD9D,QAAQ;IAMd,OAAO,CAAC,oBAAoB;IAI5B,OAAO,CAAC,WAAW;IAInB,OAAO,CAAC,cAAc;IAMtB,OAAO,CAAC,gBAAgB;IAOxB,OAAO,CAAC,kBAAkB;IAUpB,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IA+BvD,aAAa,IAAI,OAAO,CAAC,OAAO,CAAC;CA2B1C;AAED,wBAAsB,aAAa,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,CAgBzH"} {"version":3,"file":"","sourceRoot":"","sources":["file:///home/louis/Documents/spyglassmc-action/src/check_datapack.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,IAAI,MAAM,kBAAkB,CAAA;AAQxC,MAAM,WAAW,aAAa;IAC1B;;OAEG;IACH,eAAe,EAAE,OAAO,CAAA;IAExB;;OAEG;IACH,OAAO,EAAE,OAAO,CAAA;CACnB;AAkBD,qBAAa,aAAc,SAAQ,IAAI,CAAC,OAAO;IAC3C,OAAO,EAAE,MAAM,CAAA;IACf,OAAO,EAAE,aAAa,CAAA;gBAEV,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa;IAiD9D,QAAQ;IAMd,OAAO,CAAC,oBAAoB;IAI5B,OAAO,CAAC,WAAW;IAInB,OAAO,CAAC,cAAc;IAMtB,OAAO,CAAC,gBAAgB;IAOxB,OAAO,CAAC,kBAAkB;IAUpB,SAAS,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAgC3D,aAAa,IAAI,OAAO,CAAC,OAAO,CAAC;CA2B1C;AAED,wBAAsB,aAAa,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,CAgBzH"}

13
dist/index.js vendored
View File

@@ -87590,23 +87590,24 @@ class CustomService extends Service {
endLine, endColumn endLine, endColumn
}; };
} }
async checkFile(path, lang) { async checkFile(filePath, lang) {
const url = this.makeFileUrl(path); const url = this.makeFileUrl(filePath);
const content = this.getFileContent(path); const content = this.getFileContent(filePath);
this.project.onDidOpen(url, lang, 0, content); this.project.onDidOpen(url, lang, 0, content);
const docAndNode = this.project.getClientManaged(url); const docAndNode = this.project.getClientManaged(url);
if (!docAndNode) { if (!docAndNode) {
core.error(`File ${path} is not loaded`); core.error(`File ${filePath} is not loaded`);
return false; return false;
} }
const { node } = docAndNode; const { node } = docAndNode;
const errors = FileNode.getErrors(node); const errors = FileNode.getErrors(node);
if (errors.length !== 0) { if (errors.length !== 0) {
const msg = `${errors.length} error${errors.length > 1 ? "s" : ""} in ${path}`; const relPath = external_path_default().relative(this.rootDir, filePath);
const msg = `${errors.length} error${errors.length > 1 ? "s" : ""} in ${relPath}`;
if (this.options.verbose) { if (this.options.verbose) {
core.startGroup(msg); core.startGroup(msg);
for (const err of errors) { for (const err of errors) {
core.error(err.message, this.getErrorProperties(err, path, content)); core.error(err.message, this.getErrorProperties(err, filePath, content));
} }
core.endGroup(); core.endGroup();
} }

View File

@@ -1,6 +1,6 @@
{ {
"name": "spyglassmc-action", "name": "spyglassmc-action",
"version": "1.1.0", "version": "1.1.4",
"description": "Check your Minecraft datapack for syntax or schema errors using SpyglassMC", "description": "Check your Minecraft datapack for syntax or schema errors using SpyglassMC",
"main": "index.js", "main": "index.js",
"scripts": { "scripts": {

View File

@@ -125,26 +125,27 @@ export class CustomService extends core.Service {
} }
} }
async checkFile(path: string, lang: string): Promise<boolean> { async checkFile(filePath: string, lang: string): Promise<boolean> {
const url = this.makeFileUrl(path) const url = this.makeFileUrl(filePath)
const content = this.getFileContent(path) const content = this.getFileContent(filePath)
this.project.onDidOpen(url, lang, 0, content) this.project.onDidOpen(url, lang, 0, content)
const docAndNode = this.project.getClientManaged(url) const docAndNode = this.project.getClientManaged(url)
if (!docAndNode) { if (!docAndNode) {
action.error(`File ${path} is not loaded`) action.error(`File ${filePath} is not loaded`)
return false return false
} }
const { node } = docAndNode const { node } = docAndNode
const errors = core.FileNode.getErrors(node) const errors = core.FileNode.getErrors(node)
if (errors.length !== 0) { if (errors.length !== 0) {
const msg = `${errors.length} error${errors.length > 1 ? "s" : ""} in ${path}` const relPath = path.relative(this.rootDir, filePath)
const msg = `${errors.length} error${errors.length > 1 ? "s" : ""} in ${relPath}`
if (this.options.verbose) { if (this.options.verbose) {
action.startGroup(msg) action.startGroup(msg)
for (const err of errors) { for (const err of errors) {
action.error( action.error(
err.message, err.message,
this.getErrorProperties(err, path, content) this.getErrorProperties(err, filePath, content)
) )
} }
action.endGroup() action.endGroup()