Skip to content

feat: proj cost track#162

Open
wjiayis wants to merge 95 commits intomainfrom
feat/proj-cost-track
Open

feat: proj cost track#162
wjiayis wants to merge 95 commits intomainfrom
feat/proj-cost-track

Conversation

@wjiayis
Copy link
Copy Markdown
Member

@wjiayis wjiayis commented Apr 24, 2026

No description provided.

Junyi-99 and others added 30 commits January 27, 2026 02:58
@Junyi-99
refactor: update styles and prompt selection logic (#103)
37218d9
@Junyi-99
feat: initialize projectInstructions variable in conversation prepara… (
d9dbc20 
#104)
@Junyi-99
refactor: update MessageId handling to remove prefix (#105)
894532a
@kah-seng
add byok modal, update user proto and model
be7b8c8
@Junyi-99
feat: gpt-5.2, refactored streaming, improved md rendering, basic too…
50a5f0b 
…ls (#106)
@kah-seng
Merge branch 'main' of https://github.com/PaperDebugger/paperdebugger …
783f85e 
…into feat/byok
@kah-seng
update CustomModel proto, model and mapper
9603404
@kah-seng
add button functions
49ac78d
@4ndrelim
Update xtramcp tag
3ffbb5e
@Junyi-99
fix: update openaiApiKey handling to refresh model list and cache (#107)
5ea58f1
@Junyi-99
Merge branch 'staging' of https://github.com/PaperDebugger/paperdebugger
d83983b 
 into staging
@Junyi-99
refactor: remove gradient handling logic from GeneralToolCard component
74f44d5
@Junyi-99
feat: implement dark mode support and theme synchronization across co…
262f6b7 
…mponents
@kah-seng
change CustomModel in user.proto to match SupportedModel in chat.proto
38d7d56
@kah-seng
allow custom model to be selected in chat
3cb6692
@kah-seng
Merge branch 'main' into feat/byok
83efbac
@Junyi-99
fix: text patch on multi-message
49dd47f
@Junyi-99
fix: css
07321c9
@Junyi-99
fix: style
3318812
@Junyi-99
fix: style
c2d221e
@Junyi-99
add: link to report bugs
b055a17
@Junyi-99
chore: format & lint
f4e4634
@Junyi-99
docs: update qr code
e0d5808
@kah-seng
Merge branch 'main' into feat/byok
9259e76
@kah-seng
add select input, update model selection
6333acc
@kah-seng
Merge branch 'main' into feat/byok
f693cf4
@kah-seng
todo: fix gemini bad request
adf8216
@wjiayis
feat: tab completion for citation keys (#110)
6ef12a3 
#33 

In short, it'll

1. [Frontend] Recognize that user is trying to add a citation (trigger
text is `\cite{`)
2. [Frontend] Temporarily suppress default Overleaf dropdown suggestions
3. [Frontend] Get the last sentence as context for LLM
4. [Backend] Fetch bibliography in `.bib` files as raw text, and remove
irrelevant fields to save tokens
5. [Backend] Call XtraMCP to get paper abstract, using paper title as
key
6. [Backend] Query a fast LLM (hardcoded to `gpt-5.2` for now) to get at
most 3 citation keys
7. [Frontend] Suppress default Overleaf tab-completion to allow users to
accept citation suggestions
@wjiayis
chore: update helm for xtramcp (#119)
dab4e6f
@wjiayis
chore: merge branch 'main' into staging (#122)
ea701a5
Junyi-99 and others added 24 commits March 28, 2026 15:00
@Junyi-99 @4ndrelim @claude
docs: add contributing instructions (#127) (#142)
572b57d 
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>##
Co-authored-by: andre <95348273+4ndrelim@users.noreply.github.com>
Co-authored-by: Junyi Hou <junyi@xtras3.tail08d22c.ts.net>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Junyi-99 @claude
fix: set PD_MONGO_URI for in-cluster mongo and enable in_cluster (#145)
89a066b 
Co-authored-by: Junyi Hou <junyi@xtras3.tail08d22c.ts.net>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
fix as #143/#144 (development) and #140 (main).
@Junyi-99 @claude
fix: add permissions to backend caller workflows (#146)
56cbe48 
Co-authored-by: Junyi Hou <junyi@xtras3.tail08d22c.ts.net>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@wjiayis
Merge branch 'staging' into feat/proj-cost-track
9c92a9a
@wjiayis
fix: IsCustom() -> IsCustomModel
4c14874
@wjiayis
Merge branch 'staging' into feat/proj-cost-track
3dd9bd9
@wjiayis
feat: separate success cost from failed cost
862da76
@kah-seng
Add customModels to setting-store.ts
acf9653
@kah-seng
Select models by ID instead of slug
3f6871c
@kah-seng
Add loading spinner
0f9dd22
@kah-seng
Fix Copilot comments
2ff12f9
@kah-seng
fix: BYOK (#150)
56d815e 
### Key changes:
- Select models by ID instead of slug so users can have multiple API
keys for the same slug
- Add loading spinner for save/edit actions
@kah-seng
Show error when same slug and name
6508989
@kah-seng
Add tooltips
7eca4d7
@kah-seng
Add temp, parallel tools, store fields
23ea667
@kah-seng
Add tooltips
4cad7c9
@kah-seng
Allow user to configure temp, parallel tools, store params
04eb9fe
@kah-seng
Add tooltips
2a0bd5f
@kah-seng
Merge branch 'staging' into feat/byok
4774fbe
@kah-seng
Fix Copilot comments
549b5fb
@kah-seng
Merge pull request #153 from PaperDebugger/feat/byok
dcae304 
feat: BYOK enhancements
@wjiayis
Merge branch 'staging' into feat/proj-cost-track
4f8312c
@wjiayis
refactor: merge staging
60922e8
@wjiayis
chore: add test cases for failed usage
49935a2
@wjiayis wjiayis requested review from 4ndrelim and Junyi-99 April 24, 2026 12:54
@wjiayis wjiayis changed the title Feat/proj cost track feat: proj cost track Apr 24, 2026
4ndrelim
4ndrelim requested changes Apr 24, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@4ndrelim 4ndrelim left a comment

Choose a reason for hiding this comment

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

@wjiayis I think there might be some drift between your branch and main. Some of the changes in this PR e.g. in .github/workflows are already committed. Merging this branch now might make the commit history slightly messy. Also, I noticed some of Kah Seng's BYOK changes are also here. It is slightly difficult to isolate the changes introduced for cost tracking.

Could you raise a PR that only reflects your changes for cost tracking?

@4ndrelim
Copy link
Copy Markdown
Member

4ndrelim commented Apr 24, 2026 •
edited
Loading

I see the issue. Your branch is behind main because it was branched off staging, which is the intended workflow.

The problem is that staging and main had already diverged before your changes. Some changes were introduced into both branches separately, so although they are logically the same or very similar, Git treats them as different commits. This is why files such as .github/workflows/... are showing up in the diff even though they are not really part of your intended changes. This should not have been the case, but likely one of us forgot to ensure main and staging remain aligned after introducing changes separately.

Those workflow file diffs should be fine, but your branch should still not include or reflect the BYOK changes. I will fix staging over the weekend to realign it with main and make sure it can continue serving as our staging release branch. I will ensure this moving forward, but for now, i have to trouble you to ideally isolate your changes so it is easier to review and isolate.

@wjiayis
Copy link
Copy Markdown
Member Author

wjiayis commented Apr 25, 2026

@4ndrelim @kah-seng I think main doesn't have IsCustomModel yet, which this PR relies on (to determine whether to track costs or not). What should I use in alternative?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@4ndrelim 4ndrelim 4ndrelim requested changes

@Junyi-99 Junyi-99 Awaiting requested review from Junyi-99

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@wjiayis @4ndrelim @Junyi-99 @kah-seng