Skip to content

Register missing file ownership and refresh the docs snapshot - #3879

Merged
BigSimmo merged 1 commit into
mainfrom
codex/chat-psychsift-docs-registration-psychsift-docs-registration
Oct 28, 2026
Merged

BigSimmo merged 1 commit into
mainfrom
codex/chat-psychsift-docs-registration-psychsift-docs-registration

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Register the 31 tracked files that prevented the documentation pipeline from progressing. Ownership follows the feature tested or implemented, including access/privacy helpers in Data platform and UI notices in App experience.
  • Consolidate four coherent folder/name rules. Independent comparison across 10,898 tracked files confirms exactly 31 previously unplaced files gain an owner and no existing file changes owner.
  • Refresh only the generated outstanding-issues snapshot's ledger date from 26 to 27 October, matching the actual latest ledger commit. Canonical ledger contents and snapshot counts are unchanged.

Areas touched: Knowledge and records

Verification

  • Checks run:
    • node scripts/check-organisation.mjs --no-report --json — pass; 10,898 files, zero unplaced, zero ambiguous, zero findings.
    • node scripts/maintain-docs-system.mjs — passed, including its full strict read-only verification: 115 indexed roots/modules/routes, 768 npm script references, 5,933 repository links, snapshots, strict clinical hazard controls, instructions, decisions and database skills.
    • node node_modules/prettier/bin/prettier.cjs --check <seven owned paths> — all matched files use Prettier style.
    • git diff --check and normal pre-commit documentation synchronisation — passed.
  • Advisory decision-record results remain: seven warnings and 38 unanchored records, with zero decision errors. No review pins or governance checks were weakened.
  • Not run: application unit, build and browser suites; this diff changes ownership metadata and one generated provenance date, with no application behaviour change. Hosted draft checks, if published, do not establish merge readiness.

Risk and rollout

  • Risk: incorrect logical ownership or stale generated metadata; covered by independent path comparison, responsibility review and the strict documentation pipeline.
  • Rollback: a normal reviewed revert of this metadata-only commit.
  • Provider or production effects: no provider/model calls, migration, application deployment, access or billing change during the local repair. Publishing the PR starts repository CI under its existing configuration.

Clinical Governance Preflight

  • Source-backed claims still require linked source verification before clinical use
  • No patient-identifiable document workflow was introduced or expanded without explicit governance approval
  • Supabase target remains PsychSift Production (sjrfecxgysukkwxsowpy)
  • Service-role keys and private document access remain server-only
  • Demo/synthetic content remains clearly separated from real clinical sources
  • Source metadata, review status, and outdated/unknown-source behavior remain conservative
  • Deployment classification/TGA SaMD impact was checked when clinical decision-support behavior changed

Evidence / not applicable / unresolved:

  • Not applicable to this metadata-only diff: no clinical output, retrieval/ranking, access enforcement, source content, database or runtime implementation changed. Strict offline hazard validation passed; no hosted clinical/provider evaluation was performed or claimed.

Notes

  • Base: 92b3decba4c32e6b86ba97b7ea22eb34c1236425.
  • Candidate: 741affe597b3b6409ec8b70a8e3337695ae705c6.
  • Source and generated changes are limited to seven named files. The canonical issue ledger is byte-identical to the base.

Summary by CodeRabbit

  • Documentation
    • Updated system records to cover additional application, clinical-content, data-platform, delivery, and personal-practice areas, including related tests and documentation.
  • Chores
    • Updated the outstanding-issues snapshot’s recorded ledger date to October 27, 2026.

@supabase

supabase Bot commented Oct 27, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 92b3dec and 741affe.

Files selected for processing: 7
  • data/outstanding-issues-snapshot.json
  • docs/organisation/systems/answer-engine.json
  • docs/organisation/systems/app-experience.json
  • docs/organisation/systems/clinical-content.json
  • docs/organisation/systems/data-platform.json
  • docs/organisation/systems/delivery.json
  • docs/organisation/systems/personal-practice.json

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Walkthrough

This PR updates organisation-system path ownership lists, broadens several path patterns, adds related documentation, source, and test paths, and advances the outstanding-issues snapshot ledger date by one day.

Changes: Organisation ownership map

Layer / File(s) Summary
Answer and clinical content paths
docs/organisation/systems/answer-engine.json, docs/organisation/systems/clinical-content.json
The path lists add test and fixture paths. They also replace specific entries with broader patterns.
App experience paths
docs/organisation/systems/app-experience.json
The path list adds component, library, and test paths, and replaces seven front-door component entries with a wildcard pattern.
Other system paths and snapshot date
docs/organisation/systems/data-platform.json, docs/organisation/systems/delivery.json, docs/organisation/systems/personal-practice.json, data/outstanding-issues-snapshot.json
The data-platform, delivery, and personal-practice path lists add entries. The snapshot ledger date changes from 2026-10-26 to 2026-10-27.

Priority: Low

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

Change: Other

Possibly related issues

  • BigSimmo/PsychSift#3166 — The issue describes grouping outstanding issues by the organisation map. This PR adds tracked paths to that map and updates the related snapshot date.

Possibly related PRs

Suggested reviewers: claude

Merge Risk: Minimal · up to 741af

No actionable merge-blocking risk is identified in the supplied review context.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly summarizes the two main changes: registering missing file ownership and refreshing the documentation snapshot.
Description check Passed The description includes all required template sections, summarizes the changes, records verification results, explains skipped suites, documents risk and rollback, and addresses clinical governance a…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Description check

Explanation

The description includes all required template sections, summarizes the changes, records verification results, explains skipped suites, documents risk and rollback, and addresses clinical governance applicability.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (7 skipped: 7 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@gitar-bot

gitar-bot Bot commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

Copy link
Copy Markdown
Owner Author

Review record
Branch: codex/chat-psychsift-docs-registration-psychsift-docs-registration (PR #3879)
Head: 741affe
Scope: Run PR sweep
Outcome: no change needed. PR required green at head; 0 review threads; not behind main, git merge-tree against current origin/main shows no conflict. Nothing pushed.
Checks: read-only GitHub check-run, thread and merge-tree inspection only; no local gates re-run (nothing changed); no provider-backed checks run.


Generated by Claude Code

@BigSimmo
BigSimmo marked this pull request as ready for review October 28, 2026 04:34
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-28T04:35:35.186053Z 741affe Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@claude

claude Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Claude encountered an error after 2s —— View job


I'll analyze this and get back to you.

@BigSimmo
BigSimmo enabled auto-merge October 28, 2026 04:41
@BigSimmo
BigSimmo merged commit 488f7c3 into main Oct 28, 2026
62 checks passed
@BigSimmo
BigSimmo deleted the codex/chat-psychsift-docs-registration-psychsift-docs-registration branch October 28, 2026 05:02
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