Skip to content

input for sleep info#44

Merged
NoEgAm merged 10 commits intomainfrom
sleep-time
Apr 9, 2026
Merged

input for sleep info#44
NoEgAm merged 10 commits intomainfrom
sleep-time

Conversation

@tiye
Copy link
Copy Markdown
Member

@tiye tiye commented Mar 11, 2026

No description provided.

@tiye tiye requested review from a team and Copilot March 11, 2026 18:01
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 Calcit/Cumulo dependencies and snapshot, adds a new :sleep diary field surfaced in the UI, and migrates the JavaScript toolchain from a Yarn v1 lockfile to a Yarn Berry lockfile (with a new .yarnrc.yml).

Changes:

  • Upgrade Calcit tooling/dependencies (deps.cirru, @calcit/procs) and update snapshot/code to newer stdlib date/time APIs.
  • Extend diary schema/UI to capture and display a new :sleep field.
  • Switch to Yarn Berry lockfile format, add Yarn config, and remove outdated LLM guide docs.

Reviewed changes

Copilot reviewed 7 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Bumps @calcit/procs and participates in the Yarn Berry migration.
yarn.lock Regenerated from Yarn v1 to Yarn Berry lockfile (v8 metadata).
.yarnrc.yml Configures Yarn to use node-modules linker.
.gitignore Ignores Yarn .yarn/*.gz artifacts (e.g., install state).
deps.cirru Upgrades Calcit version and several Calcit/Respo/Cumulo module versions.
compact.cirru Snapshot updates + UI/schema changes for :sleep + date/time API adjustments.
Agents.md Replaces the “Related: Respo UI” link with CLI-based pointers.
llms/Respo.md Deleted (previous LLM Respo development guide).
llms/Calcit.md Deleted (previous Calcit agent guide).
Comments suppressed due to low confidence (1)

package.json:22

  • This PR switches the repo from a Yarn v1 lockfile to a Yarn Berry lockfile (__metadata.version: 8). Without pinning the Yarn version (e.g., via a packageManager field in package.json and/or yarnPath in .yarnrc.yml), different developer/CI environments may run different Yarn majors and either fail to install or regenerate yarn.lock in a different format.
  "license": "MIT",
  "dependencies": {
    "@calcit/procs": "^0.12.4",
    "copy-to-clipboard": "^3.3.3",
    "luxon": "^3.7.2",
    "shortid": "^2.2.17"
  },

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

Comment on lines +5 to +15
要求查看 Calcit 命令行工具的用法:

```bash
cr docs agents --full
```

按需查看 Respo 模块的具体用法

```bash
cr libs readme respo.calcit -f docs/Respo-Agent.md
```
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

llms/Respo.md and llms/Calcit.md were removed in this PR, but Agents.md still references ./llms/Respo.md later in the document (e.g. in the UI section / Further Reading). Please update the remaining references (or add replacement links) so the guide doesn’t contain broken links after merge.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@copilot open a new pull request to apply changes based on this feedback

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 14, 2026

@tiye I've opened a new pull request, #45, to work on those changes. Once the pull request is ready, I'll request review from you.

@NoEgAm NoEgAm merged commit 3e3f54e into main Apr 9, 2026
1 check passed
@NoEgAm NoEgAm deleted the sleep-time branch April 9, 2026 11:50
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.

4 participants