fix(boss): restore read-only job search and detail - #2291
Open
baigao417 wants to merge 2 commits into
Open
Conversation
baigao417
marked this pull request as ready for review
August 16, 2026 16:35
Author
|
CI is currently waiting on the upstream fork-workflow approval gate: CI, Security Audit, and Doc Check all report |
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.
What changed
boss searchagainst the current/web/geek/jobspage by capturing the page's ownjoblist.jsonresponse.boss detailagainst the fully rendered current job-detail page instead of the retired detail API.opencli adapter eject bossself-contained by keeping the auth helper inside the BOSS adapter directory.Strategy notes
boss searchINTERCEPTjoblist.jsonand returnszpData.jobListwithencryptJobId.security_idvalues.boss detailUI.job-primary,.job-detail,.job-boss-info,.job-sider, and.company-address.CommandExecutionError; malformed IDs fail before navigation withArgumentError.Safety boundary
boss/search:access: readboss/detail:access: readVerification
npm run build— pass, 1331 manifest entries.npm run typecheck— pass.npm run check:typed-error-lint— pass, 0 new violations.npm run check:silent-column-drop— pass, 0 new violations.opencli adapter eject boss+opencli validate boss— pass, 16 commands / 0 errors / 0 warnings.opencli browser ... verify boss/search— pass against a live logged-in browser and generated fixture.opencli browser ... verify boss/detail— pass against a live logged-in browser; 1 row, row-shape gate pass.npm pack --dry-run --ignore-scripts --json— pass; BOSS source, tests, fixture, and local auth helper are included.The complete Windows suite still has unrelated upstream path/network failures outside BOSS. The same
origin/mainbaseline reproduces 17 failures; this branch reports 18 in one run due an additional nondeterministic plugin-directory race. No BOSS test fails. Linux CI remains the authoritative full-suite gate.Release note
The current source and npm
latestremain1.8.6; this PR intentionally does not pretend that a new package has been published. After merge, the next maintainer release should include these adapters and validate the installed package with the commands listed above.