Skip to content

SQL Staging (obviously don't merge) - #106

Draft
psilabs-dev wants to merge 41 commits into
devfrom
custom-release/staging-sql
Draft

psilabs-dev wants to merge 41 commits into
devfrom
custom-release/staging-sql

Conversation

@psilabs-dev

@psilabs-dev psilabs-dev commented Mar 19, 2026

Copy link
Copy Markdown
Owner

Base Stack: #86

Apply same accepted PRs as #119.

Issues for debugging:

@psilabs-dev psilabs-dev self-assigned this Mar 19, 2026
@psilabs-dev psilabs-dev added downstream A downstream issue (not to be brought upstream). DO_NOT_MERGE Don't merge the PR. labels Mar 19, 2026
@psilabs-dev
psilabs-dev force-pushed the custom-release/staging-sql branch from 5443506 to e56aa17 Compare March 20, 2026 00:42
@psilabs-dev
psilabs-dev marked this pull request as draft March 20, 2026 00:46
@psilabs-dev
psilabs-dev force-pushed the custom-release/staging-sql branch 6 times, most recently from 3dd55c5 to b9c648c Compare March 22, 2026 05:07
@psilabs-dev
psilabs-dev force-pushed the custom-release/staging-sql branch from b9c648c to 91bdba4 Compare April 30, 2026 03:33
@psilabs-dev
psilabs-dev force-pushed the custom-release/staging-sql branch 3 times, most recently from dceca1b to 9730297 Compare August 23, 2026 01:25
@psilabs-dev
psilabs-dev force-pushed the custom-release/staging-sql branch 4 times, most recently from d33dea5 to 6abfc27 Compare September 8, 2026 05:40
psilabs-dev and others added 29 commits September 11, 2026 22:08
otherwise a tag stats call would take 10s on 100k archives/tags
The cache and cache invalidation logic is incorrect and not well-planned.
The hash was returned from inside the eval block, which returns from the
eval rather than the sub. Execution fell through to the trailing error
check, whose false branch made the sub return a one-element list, so the
edit page rendered every field blank and never redirected on an unknown
id. Assign inside the eval and return after it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019Q557x7LbNK29b9v1CqGak
# Conflicts:
#	public/js/mod/index.js
Difegue#1670's context menu predates #120's multi-category API, so the delete
handler still tested the deprecated singular selectedCategory. Under the
multi-select model that is always "", and deleting the active category
would no longer clear the filter.

Integration glue between Difegue#1670 and #120: it lives in neither branch, so a
staging rebuild cannot reproduce it by merging. Kept as an explicit commit
so future rebuilds can cherry-pick it.
@psilabs-dev
psilabs-dev force-pushed the custom-release/staging-sql branch from 6abfc27 to 5542623 Compare September 12, 2026 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO_NOT_MERGE Don't merge the PR. downstream A downstream issue (not to be brought upstream).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant