Repository navigation
Add weekly note page SmartBlocks command - #536
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: aeb1550736
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 19208ba87c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Summary
Add
<%WEEKLYNOTEPAGE:In one week%>to resolve a SmartBlocks date expression into the containing weekly page using the format configured inroam/js/weekly-notes. The command is available in any SmartBlocks workflow while Weekly Notes is enabled.DATEcommand, preserving its activeDATEBASISinstead of copying its date-basis logic.This weekthroughDATE:todayso a Monday-based week does not accidentally select the previous week.Validation
Current head:
22f36f8debc81423dfc60056277fb139096c4f4c.npm test: 22 tests passed. Expression cases exercise the installed NLP parser; command-context and graph-writing behavior still require the live check below.npm --ignore-scripts run build:roam: built with 0 errors.git diff --check: passed.tsc --noEmit: 101 errors, identical to the unchanged2f27785baseline with the same dependencies. No diagnostics in changed source files.npm ci --ignore-scriptsfailed on the existing lockfile's@esbuild/android-armplatform entry. Tests and build used the existing WorkBench dependency installation; clean-install validation remains open.Earlier live testing at
aeb15507363534e2383b775b641d7295d75cef14covered initialization, content preservation, weekly DATEBASIS, and recursion safety. That historical result does not verify the current head. Before merge, rerun the current build with explicit and DNP DATEBASIS, initialization, content preservation, recursion safety, and extension reloads.