From 8eab467d3a89a24f945fbe7d105b6d7341bc9371 Mon Sep 17 00:00:00 2001 From: Nithin Date: Sat, 29 Aug 2026 16:21:53 -0400 Subject: [PATCH] feat(explorer): add freshness sort controls --- src/fortune_intel/api.py | 5 +++- src/fortune_intel/storage/sqlite.py | 13 +++++++-- src/fortune_intel/web/api.html | 2 +- src/fortune_intel/web/app.js | 9 ++++++- src/fortune_intel/web/index.html | 11 ++++++-- src/fortune_intel/web/styles.css | 8 +++++- tests/test_public_api.py | 42 +++++++++++++++++++++++++++-- 7 files changed, 80 insertions(+), 10 deletions(-) diff --git a/src/fortune_intel/api.py b/src/fortune_intel/api.py index bc8461f..54e5286 100644 --- a/src/fortune_intel/api.py +++ b/src/fortune_intel/api.py @@ -180,6 +180,7 @@ def jobs( tier: str = Query(default="", pattern="^[A-Ea-e]?$"), opened_within_days: int = Query(default=0, ge=0, le=365), verified_within_hours: int = Query(default=0, ge=0, le=168), + sort: str = Query(default="newest", pattern="^(newest|verified)$"), status: str = Query(default="active", pattern="^(active|closed)$"), limit: int = Query(default=50, ge=1, le=50), offset: int = Query(default=0, ge=0, le=5000), @@ -191,15 +192,17 @@ def jobs( "tier": tier, "opened_within_days": opened_within_days, "verified_within_hours": verified_within_hours, + "sort": sort, "status": status, "include_synthetic": settings.show_synthetic, "us_eligibility": "eligible", } records = repository.list_jobs(**filters, limit=limit, offset=offset) + count_filters = {key: value for key, value in filters.items() if key != "sort"} return { "items": records, "count": len(records), - "total": repository.count_jobs(**filters), + "total": repository.count_jobs(**count_filters), "limit": limit, "offset": offset, "country_scope": COUNTRY_SCOPE, diff --git a/src/fortune_intel/storage/sqlite.py b/src/fortune_intel/storage/sqlite.py index 1a9a1d5..8ae5e05 100644 --- a/src/fortune_intel/storage/sqlite.py +++ b/src/fortune_intel/storage/sqlite.py @@ -241,6 +241,7 @@ def list_jobs( tier: str = "", opened_within_days: int = 0, verified_within_hours: int = 0, + sort: str = "newest", status: str = "active", limit: int = 50, offset: int = 0, @@ -282,6 +283,15 @@ def list_jobs( "j.last_seen_at IS NOT NULL AND datetime(j.last_seen_at) >= datetime('now', ?)" ) params.append(f"-{verified_within_hours} hours") + sort_order = { + "newest": "datetime(COALESCE(j.posted_at, j.first_seen_at)) DESC, j.title ASC", + "verified": ( + "datetime(j.last_seen_at) DESC, " + "datetime(COALESCE(j.posted_at, j.first_seen_at)) DESC, j.title ASC" + ), + }.get(sort) + if sort_order is None: + raise ValueError("sort must be newest or verified") params.extend([min(max(limit, 1), 200), max(offset, 0)]) with self.connect() as connection: rows = connection.execute( @@ -299,8 +309,7 @@ def list_jobs( c.name AS company_name, c.slug AS company_slug, c.ats_type FROM jobs j JOIN companies c ON c.id = j.company_id WHERE {" AND ".join(clauses)} - ORDER BY datetime(COALESCE(j.posted_at, j.first_seen_at)) DESC, - j.title ASC + ORDER BY {sort_order} LIMIT ? OFFSET ? """, params, diff --git a/src/fortune_intel/web/api.html b/src/fortune_intel/web/api.html index dd0e8e1..5f177b4 100644 --- a/src/fortune_intel/web/api.html +++ b/src/fortune_intel/web/api.html @@ -21,7 +21,7 @@

Build from evidence.

Search verified U.S. openings, inspect H-1B evidence tiers, and monitor source freshness. Responses are JSON, bounded, paginated, and rate limited.

Endpoints

-
GET /api/jobs

Search definite U.S. roles with q, company, location, tier, opened_within_days, verified_within_hours, status, limit, and offset. The opening-date filter uses employer-posted dates only; roles without a source date are excluded. The verification filter uses the most recent time OpenRole observed that individual role. Ambiguous and non-U.S. locations are never returned.

+
GET /api/jobs

Search definite U.S. roles with q, company, location, tier, opened_within_days, verified_within_hours, sort, status, limit, and offset. Use sort=newest for employer posting recency or sort=verified for OpenRole verification recency. The opening-date filter uses employer-posted dates only; roles without a source date are excluded. The verification filter uses the most recent time OpenRole observed that individual role. Ambiguous and non-U.S. locations are never returned.

GET /api/jobs/{id}

Read a privacy-limited description excerpt and the evidence used for one assessment.

GET /api/companies

Search public company identities with q, limit, and offset; inspect active-job coverage and freshness.

GET /api/coverage

Inspect the complete company denominator and auditable discovery, approval, and successful-source counts. Directory inclusion alone is not coverage.

diff --git a/src/fortune_intel/web/app.js b/src/fortune_intel/web/app.js index bde0351..e4bda86 100644 --- a/src/fortune_intel/web/app.js +++ b/src/fortune_intel/web/app.js @@ -119,6 +119,7 @@ function replaceSearchUrl(company, filters) { if (filters.verifiedWithin) { params.set("verified_within_hours", filters.verifiedWithin); } + if (filters.sort !== "newest") params.set("sort", filters.sort); const search = params.toString(); history.replaceState(null, "", search ? `/?${search}#jobs` : "/"); } @@ -139,6 +140,10 @@ function loadSearchInputs() { if (["", "6", "24", "72"].includes(verifiedWithin)) { document.querySelector("#verified-within").value = verifiedWithin; } + const sort = params.get("sort") || "newest"; + if (["newest", "verified"].includes(sort)) { + document.querySelector("#sort").value = sort; + } return params.get("company") || ""; } @@ -150,7 +155,8 @@ async function loadJobs(company = "") { const tier = document.querySelector("#tier").value; const openedWithin = document.querySelector("#opened-within").value; const verifiedWithin = document.querySelector("#verified-within").value; - const filters = { query, location, tier, openedWithin, verifiedWithin }; + const sort = document.querySelector("#sort").value; + const filters = { query, location, tier, openedWithin, verifiedWithin, sort }; replaceSearchUrl(company, filters); if (query) params.set("q", query); if (company) params.set("company", company); @@ -158,6 +164,7 @@ async function loadJobs(company = "") { if (tier) params.set("tier", tier); if (openedWithin) params.set("opened_within_days", openedWithin); if (verifiedWithin) params.set("verified_within_hours", verifiedWithin); + if (sort !== "newest") params.set("sort", sort); const list = document.querySelector("#job-list"); const empty = document.querySelector("#empty-state"); diff --git a/src/fortune_intel/web/index.html b/src/fortune_intel/web/index.html index 7d2a232..db1cfaf 100644 --- a/src/fortune_intel/web/index.html +++ b/src/fortune_intel/web/index.html @@ -6,7 +6,7 @@ OpenRole Intelligence - +
@@ -70,6 +70,13 @@

Find roles with a clearer
sponsorship signal.

+
@@ -127,6 +134,6 @@

A signal you can inspect.

Evidence-first job intelligence Trust & methodology · Built for transparency, freshness, and correction. - + diff --git a/src/fortune_intel/web/styles.css b/src/fortune_intel/web/styles.css index b51cd81..51f1ddf 100644 --- a/src/fortune_intel/web/styles.css +++ b/src/fortune_intel/web/styles.css @@ -61,7 +61,7 @@ h1 { margin-top: 20px; font-size: clamp(43px, 6vw, 76px); line-height: 1.04; fon h1 em { color: var(--green); font-style: normal; } .hero > p { max-width: 680px; margin: 24px auto 0; color: var(--muted); font-size: 17px; line-height: 1.65; } -.search-panel { max-width: 1400px; margin: 40px auto 0; padding: 9px; display: grid; grid-template-columns: 1.25fr 1fr 1fr .9fr .9fr auto; gap: 0; background: var(--card); border: 1px solid var(--line); border-radius: 15px; box-shadow: 0 20px 60px rgba(32,48,39,.11); text-align: left; } +.search-panel { max-width: 1500px; margin: 40px auto 0; padding: 9px; display: grid; grid-template-columns: 1.15fr 1fr 1fr .9fr .9fr .8fr auto; gap: 0; background: var(--card); border: 1px solid var(--line); border-radius: 15px; box-shadow: 0 20px 60px rgba(32,48,39,.11); text-align: left; } .search-panel label { padding: 8px 18px; border-right: 1px solid var(--line); } .search-panel label span { display: block; margin-bottom: 5px; font-size: 11px; font-weight: 700; text-transform: uppercase; letter-spacing: .08em; color: #808a85; } .search-panel input, .search-panel select { width: 100%; border: 0; outline: 0; background: transparent; color: var(--ink); } @@ -150,6 +150,12 @@ td { font-size: 14px; } .pagination button { min-height: 40px; } .pagination button:disabled { opacity: .4; cursor: not-allowed; } +@media (max-width: 1200px) { + .search-panel { max-width: 920px; grid-template-columns: repeat(3, minmax(0, 1fr)); } + .search-panel label:nth-of-type(3n) { border-right: 0; } + .search-actions { grid-column: 1 / -1; grid-template-columns: 1fr auto; min-height: 46px; } +} + @media (max-width: 850px) { nav { display: none; } .search-panel { grid-template-columns: 1fr; } diff --git a/tests/test_public_api.py b/tests/test_public_api.py index b68e37e..c575076 100644 --- a/tests/test_public_api.py +++ b/tests/test_public_api.py @@ -103,6 +103,42 @@ def test_job_api_can_filter_by_the_last_time_a_role_was_verified(tmp_path): assert client.get("/api/jobs", params={"verified_within_hours": 169}).status_code == 422 +def test_job_api_can_sort_by_the_last_time_a_role_was_verified(tmp_path): + client, app = make_client(tmp_path) + company_id = app.state.repository.upsert_company("Verification Sort Check") + job_ids: dict[str, str] = {} + for external_id in ("old", "new"): + job_ids[external_id] = app.state.repository.upsert_job( + company_id, + JobRecord( + company_name="Verification Sort Check", + title=f"Verification sort {external_id}", + url=f"https://jobs.example.test/verification-sort/{external_id}", + source="verification-sort-test", + external_job_id=external_id, + location="Austin, TX", + ), + assess_sponsorship(""), + ) + with app.state.repository.connect() as connection: + connection.execute( + "UPDATE jobs SET last_seen_at = ? WHERE id = ?", + ((datetime.now(UTC) - timedelta(hours=4)).isoformat(), job_ids["old"]), + ) + + response = client.get( + "/api/jobs", + params={"q": "Verification sort", "sort": "verified"}, + ) + + assert response.status_code == 200 + assert [item["title"] for item in response.json()["items"]] == [ + "Verification sort new", + "Verification sort old", + ] + assert client.get("/api/jobs", params={"sort": "oldest"}).status_code == 422 + + def test_public_readiness_uses_lightweight_checks(tmp_path, monkeypatch): client, app = make_client(tmp_path) called = {} @@ -256,13 +292,15 @@ def test_dashboard_hidden_empty_state_cannot_display_with_results(tmp_path): assert "[hidden] { display: none !important; }" in stylesheet assert "empty.hidden = true;" in script assert "empty.hidden = data.items.length !== 0;" in script - assert "/assets/styles.css?v=9" in dashboard + assert "/assets/styles.css?v=10" in dashboard assert 'id="opened-within"' in dashboard assert 'id="verified-within"' in dashboard + assert 'id="sort"' in dashboard assert 'id="clear-filters"' in dashboard - assert "/assets/app.js?v=10" in dashboard + assert "/assets/app.js?v=11" in dashboard assert 'params.set("opened_within_days", openedWithin);' in script assert 'params.set("verified_within_hours", verifiedWithin);' in script + assert 'params.set("sort", sort);' in script assert "function replaceSearchUrl(company, filters)" in script assert "function loadSearchInputs()" in script