Skip to content

feat: improve admin, add "User"#69

Closed
juyu0617 wants to merge 0 commit into
mainfrom
feature/django_admin
Closed

feat: improve admin, add "User"#69
juyu0617 wants to merge 0 commit into
mainfrom
feature/django_admin

Conversation

@juyu0617
Copy link
Copy Markdown
Contributor

@juyu0617 juyu0617 commented Jul 27, 2023

add sorting of custom calling objects(created and updated), add model "User"; make the get_user_from_token more readable by sending nickname instead of "sub"
image
image

Known shortages

  1. "collection id" still can not be sorted.
  2. model "User" now is independent from other models and need change

@juyu0617 juyu0617 requested a review from iziang July 27, 2023 08:00
@apecloud-bot apecloud-bot added the size/M Denotes a PR that changes 30-99 lines. label Jul 27, 2023
iziang
iziang reviewed Jul 27, 2023
View reviewed changes
Comment thread kubechat/auth/validator.py Outdated
Copy link
Copy Markdown
Contributor

@iziang iziang Jul 27, 2023

Choose a reason for hiding this comment

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

payload["sub"] is required as it's the user's unique identifier.

iziang
iziang reviewed Jul 27, 2023
View reviewed changes
Comment thread kubechat/models.py Outdated
Copy link
Copy Markdown
Contributor

@iziang iziang Jul 27, 2023

Choose a reason for hiding this comment

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

where are the User objects being created?

Copy link
Copy Markdown
Contributor Author

@juyu0617 juyu0617 Jul 27, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

haven't changed yet. I think "User" should be created when creating a collection, but this may lead to great changes in function "get_user" which is used for 17 times in views.py. So i'm still thinking the right way to fix it. Maybe define a new function.

Copy link
Copy Markdown
Contributor

@iziang iziang Jul 28, 2023

Choose a reason for hiding this comment

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

If it's not ready for review, you can change this to a draft PR.

@juyu0617 juyu0617 marked this pull request as draft July 28, 2023 02:18
@apecloud-bot apecloud-bot added size/L Denotes a PR that changes 100-499 lines. and removed size/M Denotes a PR that changes 30-99 lines. labels Jul 28, 2023
@juyu0617 juyu0617 force-pushed the feature/django_admin branch from 4bec9d2 to f78f826 Compare August 1, 2023 06:44
@iziang iziang closed this Nov 5, 2023
@iziang iziang deleted the feature/django_admin branch December 1, 2023 07:34
@earayu earayu mentioned this pull request Apr 25, 2026
Merged
earayu added a commit that referenced this pull request Apr 25, 2026
@earayu @claude
docs(modularization): add agent-message-protocol-design.md (#69 / D8.…
fb0abcb 
…0) (#1691)

* docs(modularization): add agent-message-protocol-design.md (#69 / D8.0)

Pure doc landing per architect msg=27b4b20a in #68 thread. Verbatim
copy of the consolidated design content from 符炫炜 — no edits, no
implementation changes.

This is the canonical SSoT for the Phase 8 D8 protocol redesign:
chat protocol, message storage, FE/BE wire format, and the
must-delete legacy field disposition. PM will split D8.1–D8.7
implementation lanes after @earayu2 reviews the direction.

Ghost-check: none — doc-only landing, no code or contract change.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* docs(modularization): add §2.4 tool naming convention amendment per architect msg=ca153960

Verbatim 1-paragraph addition from 符炫炜 #68 thread amendment, ensuring
MCP-namespace-ready tool naming so D9 (MCP server interface) doesn't
require wire schema rename.

Ghost-check: none — pure additive doc amendment, no scope creep.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* docs(modularization): add §2.5 web-backend MCP-capable agent target architecture amendment

Verbatim copy of architect msg=（§2.5 verbatim block）— Weston msg=6418b28b
contribution covering FE-doesn't-know-MCP boundary, backend agent runtime
projection, tool boundary, and D9 design deliverables list.

Pure additive. Ghost-check: none — doc-only, no scope creep.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* docs(modularization): add PydanticAI compatibility note per PM msg=d72f9de9

Verbatim 1-line implementation note inside Standards Survey Result —
confirms wire choice (AI SDK v5) remains compatible with future
PydanticAI-backed backend agent runtime via UIAdapter integration.

Pure additive. Ghost-check: none.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
This was referenced Apr 25, 2026
Merged
Merged
Merged
earayu added a commit that referenced this pull request Apr 30, 2026
@earayu
docs(task-61): DB adapter compat spec v1 — vector + graph cross-impl …
ed8def2 
…audit (#1928)

* docs(task-61): DB adapter compat spec v1 — vector + graph cross-impl audit

Architect spec v1 起草 per earayu2 directive (msg=8b989470 / msg=2bad8e75
/ msg=f26b703e) + PM 不穷 task #72 dispatch.

Streaming evidence integration from 8 lanes:
- huangheng msg=ed2f2973: 3 vector P0 candidates (cross-tenant /
  filter silent / collection init)
- Bryce msg=8e895471 task #69: 11 vector findings (4 P0 + 3 P1 + 4 P2,
  including upgraded score normalization P0-V3/V4)
- 冬柏 msg=3e93bb64 task #67: 3 missing Protocol method tests
  (bulk_upsert_entity_with_lineage_parts P0 + remove_relation_lineage
  P1 + list_entities P1)
- chenyexuan msg=f298011e + PR #1926: workflow paths filter dead
  reference P0-W1 (in flight)
- cuiwenbo msg=dfebf706 task #70: FE/UX 3 candidates (score, viz error
  vs empty, confidence_score)
- Planetegg msg=db7fb085 + msg=41906f4 + msg=41665d7e task #65: alias
  resolution gather P2-S1 + Singapore QDRANT_MULTITENANT=True (no
  hot-fix needed) + env shape verify
- ziang task #64 graph store audit (in_progress, will fold-in)
- dongdong task #71 deploy/typed schema (in_progress, will fold-in)

Spec structure:
- §1 inventory by lane with file:line evidence
- §2 缺口 by severity (P0 CRITICAL hot-fix candidate / P0 必修 / P1
  允许差异 declare / P2 性能优化 / YAGNI)
- §3 三层 design direction per Weston msg=85e527e3 framework
- §4 sub-task dispatch (Phase A 8 lane parallel + Phase B per-P0
  three-PR-pattern + Phase C P2 + Phase D PR #1926 unblock)
- §5 acceptance: P0/P1 standards + boundary test gate + e2e + sample
  limitation免责
- §6 CR mandatory checklist citing Lesson #11-#16 family from
  PR #1916/#1924/#1922 sediment + new Lesson #16 candidate (workflow
  paths dead reference)

Sample limitation: spec evidence from streaming surface, not
huangzhangshu collected gap list — fix-forward amend after
huangzhangshu lane completes + Bryce/ziang audit slice输出.

Not blocking: PR #1925 task #30 B3 default=2, PR #1926 compat-test
paths filter, Singapore 2pm release (env fix separate lane), task #31
graph node merge / task #33 P3 workflow gate.

* docs(task-61): fix-forward Weston BLOCKER + 5 streaming integration

Weston msg=13dd5e91 BLOCKER (score normalization severity drift):
保持 P0-V3+V4 P0 across §1.1 / §2.2 / §5.3 — score 方向是 caller
语义硬契约，不能在 PGVector/Qdrant 间显示反向。§2.2 加 P0-V3+V4
显式行 + §5.3 加 test_score_normalization_in_vector.py boundary
test (跨 metric × 跨 adapter 全 6 cell parametrize).

Streaming integrations (5 lane):

1. Bryce msg=23a2f514 P0-V1 first-principles 重新定性 — Qdrant
   legacy mode tenant isolation 是 collection name level 不是 query
   filter level (verify qdrant_connector.py:442-446)，下沉 P1-V4
   defense-in-depth (legacy mode deprecation follow-up 候选).

2. Bryce msg=8e895471 11 vector findings — 4 P0 (cross-tenant
   下沉 / filter silent / score V3+V4) + 3 P1 (collection init /
   batch atomicity / filter Or 语义) + 4 P2.

3. dongdong msg=4201465a + PR #1929 + cuiwenbo msg=bcec38ad —
   P0-D1 Helm worker Neo4j env missing (Singapore graph viz
   root-cause); P1-D1 e2e shape matrix gap; P1-D2 Nebula no Helm
   first-class; P1-D3 typed schema 缺 vector backend exposure.

4. chenyexuan NIT — Lesson #16 candidate cite added §6.

5. Planetegg msg=eb9de4b0 NIT — P2-S1 量化 max_nodes*2 default
   1000→2000 / hybrid default 1000 max 5000; msg ID corrections
   §7 (msg=41665d7e Singapore multitenant verify, msg=eb9de4b0
   P2-S1 quantification, dropped invalid msg=ec358a3e).

冬柏 PR #1927 commit b2234ae fold-in §5.3 (38 cases incl
zero-side-effect + replay idempotency post-NIT).

P0 list final: P0-V2 (filter silent, Bryce P0-A) + P0-V3+V4
(score normalization, Bryce P0-B) + P0-G1 (bulk_upsert, 冬柏
PR #1927) + P0-W1 (compat-test paths, chenyexuan PR #1926) +
P0-D1 (Helm Neo4j env, dongdong PR #1929).

* docs(task-61): § 3.1.1 historical residue cleanup per Weston msg=fdf04a69 NIT — strike old P0 hot-fix path (P0-V1 已下沉 P1-V4 per Bryce first-principles verify)

* docs(task-61): final consistency cleanup per Weston msg=e414d3cf — line 14 count 4+3+4 to 3 P0 + 4 P1 + 4 P2; § 5.1 P0-V1 line removed; § 5.2 P1-V4 defense-in-depth boundary test added
@earayu earayu mentioned this pull request Apr 30, 2026
Merged
3 tasks
earayu added a commit that referenced this pull request Apr 30, 2026
@earayu @claude
docs(cr-checklist): task #61 + task #30 B3 close-out sediment fold-in (…
dc79aad 
…#1932)

§ 四 加 8 lesson sediment（task #30 B3 + task #61 全 P0 闭环累计实证）+ § 六 sediment 引用追加 6 PR commit cross-link + § 八 修订记录追加本 PR fold trail。

新增 lesson:
- Lesson #12 v7.4: external API raw contract verify (task #61 P0-B PR #1930
  Qdrant euclid raw direction first-application + fix-forward 1e30a00)
- Lesson #12 v8 second-application: test docstring fake guardrail (task #61
  P0-G1 PR #1927 description_parts assertion 缺位 fix-forward 1953933)
- Lesson #12 v9: first-principles verify catch surface signal mistakes
  (task #61 P0-V1 重新定性 Bryce + task #61 P0-B Qdrant euclid Weston catch
  双独立 source 同源 first/second-application)
- Lesson #13 v2.3: deploy manifest dual-side rewrite (task #61 P0-D1 PR #1929
  Helm Neo4j worker env first-application)
- Lesson #13 v3 application demo 2: cross-source default value alignment
  (task #30 B3 PR #1925 commit dae43f5 三 source 同步 first-application)
- Lesson #14 application demo: spec 内部 default 漂浮 multi-iteration cleanup
  (task #30 B3 PR #1925 fix-forward dae43f5 § 3.1.1 line 85 cleanup
  second-application demo, first-application 在 task #35 6 轮 fix-forward)
- Lesson #16: CI workflow paths filter dead reference 反 pattern (task #61
  P0-W1 PR #1926 first-application demo + Lesson #15 file-move 3-step verify
  升级到 v2 4-step grep .github/workflows/*.yml paths 同步)
- Lesson #17: backend 收敛 contract 优于上层 fork (simple-stable + private-deploy
  paramount directive earayu2 msg=1224bec8 在 cross-adapter contract 设计时
  应用; task #69 P0-B + task #70 P1 候选 1 cross-PR 一次性收敛 first-application)

跨 PR 多独立 source 同源 catch trail:
- Lesson #12 v9: Bryce msg=23a2f514 + Weston msg=86e05a8e 双独立 source
- Lesson #16: chenyexuan msg=f298011e + 冬柏 msg=3e93bb64 双独立 source
- Lesson #17: cuiwenbo msg=cedc7703 + Bryce msg=9895a148 双独立 source
- Lesson #13 v3 application demo 2: huangheng msg=bf785b12 + Planetegg
  msg=c63acbf5 + Weston msg=1e6b0838 三独立 source

per architect msg=c4cdf634 + msg=daaeeab5 + msg=03c892e0 sediment dispatch.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@iziang iziang iziang left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

size/L Denotes a PR that changes 100-499 lines.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@juyu0617 @iziang @apecloud-bot