Repository navigation
feat: bring every example onto Lumin's current tool surface - #1
Merged
Merged
Conversation
The result block carries only tool_use_id, so every logged call read "unknown"; the name now comes from its mcp_tool_use. The snapshot is regenerated from a live tools/list, and the root docs drop the renamed _advanced names, which stop working after one release.
… number intuition Lumin now rejects a place of judgement without judgement_datetime, which failed live runs; the route fixes the moment and every tool takes the judgement_* fields. Lost questions use the four lost-article step tools, and the number check shows its methods and both ruling-planet sets.
…to finish Renamed and bundled tools move to get_csl_condition, get_surgery_dasha_periods, get_chronic_recovery and get_malignancy_signatures; get_paradigm_panel replaces the tallying verdict. A live run was cut off at 24000 tokens, so the budget is the model's maximum.
get_compatibility_advanced becomes get_couple_csl_factors and get_couple_dasha_sync, with no overall score. Ashta Koota now scores the male chart as the groom, as the tool defines it.
The channel scores are raw signed numbers, not 0 to 100, and the levels and outdoor rating were the model's own. The season banner reads the 4th-cusp step tool, each window shows Sapta Nadi, and the current reading is dated.
…enamed tool, read to the last page
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.
Brings all nine examples onto Lumin's current tool surface. Every route was run live against the real server (Sonnet 5.5) and returned 200 with
toolsFailed=0.Why
A first live pass of all 12 routes found two failures:
/api/ask: Lumin now rejects a place of judgement withoutjudgement_datetime. The prompt said "Moment of judgment: now" and documented the old birth-style fields, so the call failed. The route now fixes the moment server-side and every horary tool takes thejudgement_*fields.The server also renamed twelve
_advancedtools (old names work for one more release) and split bundles into step tools. This PR moves every app off the old names and onto step tools where an app reads one block.Per app
judgement_*fields everywhere, includingget_horary_serialsessions.get_horary_number_intuitionreplaces the renamed validator. Lost questions use the four lost-article step tools. New cards: the number methods side by side, and both ruling-planet definitions.get_csl_condition,get_surgery_dasha_periods,get_chronic_recovery,get_malignancy_signaturesreplace bundles.get_paradigm_panelreplaces the tallyingget_multi_system_verdict.get_couple_csl_factors+get_couple_dasha_syncreplaceget_compatibility_advanced, with no overall score (the old one was an unlabelled average). Adds Punarphoo and KSK's discord rule. Ashta Koota now scores the male chart as the groom, as the tool defines it.get_weather_csl_fourth; each window shows Sapta Nadi.find_auspicious_triangulationreplacesget_muhurta_advanced, read to the last page.get_ruling_planets_at_judgement.get_nakshatra_padaandget_shadbala_index; the 0 to 100 strength is labelled as Lumin's index, not the BPHS Shadbala.run_pre_verdict_auditnow matches what the tool computes.Shared: the run log named every tool
unknown(the result block carries onlytool_use_id); it now names each one.scripts/tool-names.jsonis regenerated from a livetools/list(240 to 315). Root docs drop the renamed names.Checks
bun run check:tools: no unknown names (315 known)bun run typecheck: all ten packages exit 0bun run build: all nine apps exit 0toolsFailed=0. today and elect re-run on fresh production servers after the last edits.Known open items
🤖 Generated with Claude Code