Skip to content

chore: fix duplicate package.json field, update gitignore and docs #66

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.

Sign up for GitHub

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

Merged
glandua merged 1 commit into from
Apr 10, 2026
Merged

chore: fix duplicate package.json field, update gitignore and docs #66

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,18 @@ node_modules/
# Local-only context (LLM analyses, workspace archives, private notes)
.local/

# Python
__pycache__/
*.pyc
*.pyo
.venv/
*.egg-info/
dist/
build/

# Rust/CosmWasm build artifacts
target/
*.wasm

# Claude Code
CLAUDE.local.md
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ This project is developed in collaboration with Regen Network community. Contrib

## License

[To be determined - suggest Apache 2.0 or MIT for code, CC-BY-SA for documentation]
Code is licensed under [Apache License 2.0](LICENSE). Documentation is licensed under [CC BY-SA 4.0](LICENSE-CC-BY-SA). See [LICENSE.md](LICENSE.md) for details on which license applies to which files.

---

Expand Down
10 changes: 10 additions & 0 deletions docs/DEV_SETUP.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ npm install
|--------|---------|---------|
| `verify` | `node scripts/verify.mjs` | Validate required files, schemas, mechanism index |
| `build:index` | `node scripts/build-mechanism-index.mjs` | Regenerate mechanism index in README.md |
| `verify:m010:datasets` | `node scripts/verify-m010-datasets.mjs` | Validate M010 dataset fixtures |
| `check:index` | `node scripts/build-mechanism-index.mjs --check` | Verify mechanism index is current (CI-safe) |

---
Expand All @@ -103,6 +104,15 @@ node mechanisms/m010-reputation-signal/reference-impl/m010_score.js
# Test vectors are in reference-impl/test_vectors/
# Input: vector_v0_sample.input.json
# Expected: vector_v0_sample.expected.json

# Run M012 supply algorithm self-test
node mechanisms/m012-fixed-cap-dynamic-supply/reference-impl/m012_supply.js

# Run M013 fee routing self-test
node mechanisms/m013-value-based-fee-routing/reference-impl/m013_fee.js

# Run M015 contribution rewards self-test
node mechanisms/m015-contribution-weighted-rewards/reference-impl/m015_score.js
```

### Adding New Test Vectors
Expand Down
1 change: 0 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
"private": true,
"type": "module",
"version": "0.0.0",
"type": "module",
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot Mar 25, 2026

Choose a reason for hiding this comment

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

high

The package.json file contains a duplicate "type": "module" field. While some parsers might be lenient, having duplicate keys in a JSON object is invalid and can lead to unpredictable behavior. Removing the duplicate entry ensures the file is well-formed and correctly parsed.

Copy link
Copy Markdown
Contributor Author

@brawlaphant brawlaphant Mar 31, 2026

Choose a reason for hiding this comment

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

Thanks for flagging — this PR is the fix for that exact issue. The diff shows the duplicate "type": "module" being removed (the - line). After this PR merges, package.json will have exactly one "type": "module" field on line 4.

"scripts": {
"verify": "node scripts/verify.mjs",
"verify:m010": "node scripts/verify-m010-reference-impl.mjs",
Expand Down