Skip to content

chore: add MIT license, ignore node_modules, fix runner scripts - #1

Merged
c0d3ster merged 3 commits into
mainfrom
chore/update-metadata-and-license
Aug 1, 2026
Merged

chore: add MIT license, ignore node_modules, fix runner scripts#1
c0d3ster merged 3 commits into
mainfrom
chore/update-metadata-and-license

Conversation

@c0d3ster

@c0d3ster c0d3ster commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add node_modules/ to .gitignore
  • Rewrite package.json description to better convey what nightlight does
  • Fix plan/overnight npm scripts to invoke via bash explicitly (cross-platform/Windows friendly)
  • Add MIT LICENSE

Test plan

  • N/A — metadata/config only, no runtime behavior to exercise beyond npm run plan / npm run overnight invoking correctly

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Added an MIT License with usage, distribution, and warranty terms.
    • Updated the package description to clarify unattended session and permission-control capabilities.
  • Chores

    • Improved repository exclusions for installed dependencies.
    • Updated command scripts for more consistent execution.

- add node_modules/ to .gitignore
- update package.json description
- fix plan/overnight scripts to invoke via bash explicitly
- add MIT LICENSE
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • .claude/skills/plan-tasks.md
⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 610688e0-a359-46cc-ae6b-f2bb17657019

📥 Commits

Reviewing files that changed from the base of the PR and between 9b2d42d and 12c275f.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • .claude/skills/plan-tasks.md

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The project adds an MIT license, ignores .env and node_modules/, and updates the package description and plan/overnight scripts to use bash.

Changes

Project updates

Layer / File(s) Summary
Project configuration and package commands
.gitignore, LICENSE, package.json
.gitignore excludes .env and node_modules/. LICENSE adds the MIT License. package.json updates the description and invokes the plan and overnight scripts through bash.

Estimated code review effort: 2 (Simple) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main changes: adding the MIT license, ignoring node_modules, and fixing the runner scripts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/update-metadata-and-license

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@c0d3ster
c0d3ster merged commit c724cc8 into main Aug 1, 2026
1 check passed
c0d3ster added a commit that referenced this pull request Aug 3, 2026
* chore(tasks): number and stack self-targeting task list

Assign #1-#10 task numbers and [stack] annotations to TASKS.md's
Agent-Ready/Verify/Research/Decisions items for nightlight's
self-improvement backlog. Introduce docs/nightlight-meta.json with
nextTaskNumber: 11.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* chore: gitignore .claude/worktrees/

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
c0d3ster added a commit that referenced this pull request Aug 8, 2026
- Archive #2 (fresh-subprocess-per-task execution model, PR #24) as
  complete; NEEDS HUMAN cache-read comparison trial ongoing outside
  TASKS.md tracking, see archive note
- Add #12 [stack: solo], bundling four turn/cache-read efficiency
  findings from the c0d3ster cost audit, placed at the top of
  Agent-Ready
- #1 confirmed not implemented, left open and untouched
- nextTaskNumber 12 -> 13, tasksCompleted 0 -> 1

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant