docs(jules): curate agent journals, canonicalize .jules casing, remove scratch debris#626
Open
seonghobae wants to merge 9 commits into
Open
docs(jules): curate agent journals, canonicalize .jules casing, remove scratch debris#626seonghobae wants to merge 9 commits into
seonghobae wants to merge 9 commits into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Documentation-only update to the repository’s .Jules guidance notes, consolidating recent performance, accessibility, and security learnings for future reviewers/agents.
Changes:
- Replaced
.Jules/sentinel.mdentry with guidance on hardening Pydantic string validation against control characters. - Updated
.Jules/palette.mdaccessibility notes (disabled button guidance, table-row input labeling, abbreviation announcements). - Expanded
.Jules/bolt.mdwith a larger set of performance lessons and action patterns.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .Jules/sentinel.md | Updates security guidance for validating user-provided strings (control characters). |
| .Jules/palette.md | Refreshes accessibility guidance for UI patterns and screen reader behavior. |
| .Jules/bolt.md | Adds/refreshes performance optimization learnings and recommended patterns. |
Comments suppressed due to low confidence (1)
.Jules/bolt.md:13
- The guidance here duplicates the later entry “2024-07-01 - Avoid O(N^2) Complexity in Graph Exporters” (both advise precomputing lookup Maps/Sets to avoid nested
.find()/.some()loops). Keeping both makes the notes harder to scan and increases the chance future edits diverge. Consider consolidating into a single entry (or cross-referencing one from the other).
## 2026-06-20 - O(N^2) loops for finding items in export
**Learning:** Nested array `.find()` iterations within loops parsing graph connections result in O(N^2) complexity, significantly degrading UI performance for large outputs.
**Action:** Always pre-compute a lookup `Map` in O(N) when multiple specific node lookups are needed within iterative processes.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
seonghobae
enabled auto-merge (squash)
July 24, 2026 07:26
Rewrite .jules/{bolt,palette,sentinel}.md as deduplicated, newest-first
journals. All entry dates are now derived from first-appearance evidence
in git history (git log -S); the previous 2024/2025 dates predated the
repository and were fabricated. Merge overlapping entries (export lookup
maps, map.set patterns, abbreviation badges, disabled-control semantics,
STRIX scope guidance), translate the remaining Korean entries to English,
resolve the hot-path-vs-batch Map guidance contradiction with explicit
cross-references, absorb the backend HKDF sentinel fragment, and restore
the share-redaction sentinel entry whose heading on main was a literal
unexpanded date command. AGENTS.md now documents the journal format and
the evidence-based dating rule.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Its single HKDF entry now lives in the root .jules/sentinel.md journal; agents must not grow per-package journal copies. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Delete fix_plan3-15.txt planning debris from the repo root and add the CodeGraph index directory to .gitignore; both are agent working artifacts that never belonged in version control. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Delete the capitalized .Jules/ index entries that collided with the canonical lowercase .jules/ on case-insensitive filesystems (Windows, default macOS), causing phantom git add failures and divergent journal content. AGENTS.md now documents .jules/ as the only allowed path. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Extend the project_name, member_subject, and conn_name validation patterns from C0+DEL to the full C0/C1 control ranges so the implemented validation matches the sentinel journal guidance on log/terminal escape injection, with C1 rejection tests. Also fix the backend/.jules casing example in AGENTS.md flagged in review. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
seonghobae
enabled auto-merge
July 24, 2026 08:44
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The
.Julesagent journals had accumulated systemic quality defects: the repo tracked BOTH.Jules/and.jules/as case-duplicate directories with divergent content (colliding on case-insensitive filesystems and causing phantomgit addfailures), a third fragment lived atbackend/.jules/sentinel.md, most entry dates were fabricated (2024/2025 dates predate the repository, whose history starts 2026-02), several entries were duplicated or contradictory, and 13fix_plan*.txtagent-scratch files were tracked at the repo root.What
.Jules/directory;.jules/(lowercase) is now the single canonical journal path, documented in AGENTS.md along with the journal entry format and an evidence-based dating rule.bolt.md(14 entries),palette.md(13), andsentinel.md(3) as deduplicated, newest-first journals. Every date is derived from first-appearance evidence in git history (git log -S). Overlapping entries merged (export lookup maps,map.setpatterns, abbreviation badges, disabled-control semantics, STRIX scope guidance), Korean entries translated to English, and the hot-path-vs-batchMapguidance contradiction resolved with explicit cross-references between the two entries.backend/.jules/sentinel.md(HKDF entry) into the root sentinel journal; restore the share-redaction sentinel entry whose heading onmainwas a literal unexpanded$(date +%Y-%m-%d).fix_plan3-15.txtscratch files; add.codegraph/to.gitignore.backend/app/schemas.pyonly rejected C0+DEL. Theproject_name,member_subject, andconn_namepatterns now reject the full C0/C1 ranges (^[^\x00-\x1F\x7F-\x9F]+$), with C1 rejection cases added tobackend/tests/test_schema_validation.py. Names containing C1 bytes (U+0080-U+009F) are now rejected with 422; legitimate names are unaffected.Review notes