Jakob Wennberg
ec27228a8e
Em dashes (—) and en dashes (–) had spread across comments, docs, tests, and a few UI strings, reading as AI-generated boilerplate rather than house style. Replaced each with punctuation matching its context: colon for explanatory clauses, comma for asides, plain hyphen for numeric/legal ranges (e.g. "21-23§"), "to"/"till" for date ranges, parentheses for paired-dash asides. messages/en.json and messages/sv.json were fixed by hand together to keep sv/en in sync. Left untouched where the dash is the functional subject rather than decorative punctuation: date-range-parser.ts's separator regex, charset-repair.ts's CP1252 byte-mapping table (and its test), the SIE encoding mojibake docs, generic-csv.ts's minus-sign normalizer, the agent system-prompt files that already instruct against em dashes, and a golden iXBRL test fixture compared byte-for-byte. Also fixes two bugs surfaced along the way: an off-by-one in ApiKeysPanel's scope-label split (a leftover from an earlier partial pass), and a charset-repair test that had lost the literal en-dash it exists to verify. Regenerated the agent atom seed migration (skills:generate) since 27 SKILL.md files changed. Added a CLAUDE.md rule against em/en dashes, with an explicit carve-out for the functional-dash cases above. Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
85 lines
3.9 KiB
TypeScript
85 lines
3.9 KiB
TypeScript
'use client'
|
|
|
|
import { useTranslations } from 'next-intl'
|
|
import { useCompany } from '@/contexts/CompanyContext'
|
|
import { useAgentSheet } from '@/components/agent/AgentSheetProvider'
|
|
import { ENABLED_EXTENSION_IDS } from '@/lib/extensions/_generated/enabled-extensions'
|
|
|
|
export type SettingsGroupKey = 'account' | 'company' | 'accounting' | 'sales' | 'tools'
|
|
|
|
export interface SettingsNavItem {
|
|
id: string
|
|
href: string
|
|
label: string
|
|
group: SettingsGroupKey
|
|
}
|
|
|
|
export interface SettingsNavGroup {
|
|
key: SettingsGroupKey
|
|
label: string
|
|
items: SettingsNavItem[]
|
|
}
|
|
|
|
// Rail group order: personal first (Konto), then company-scoped buckets.
|
|
const GROUP_ORDER: SettingsGroupKey[] = ['account', 'company', 'accounting', 'sales', 'tools']
|
|
|
|
/**
|
|
* Single source of truth for the settings sections, their conditional
|
|
* visibility, and their grouping. Consumed by both the full-page rail and the
|
|
* routed settings modal so the two can never drift on which sections show for
|
|
* AB vs EF, sandbox, identity-verified, or enabled extensions.
|
|
*
|
|
* Visibility is derived from client context (no extra fetch): `isSandbox`
|
|
* comes from CompanyContext, identity from the agent sheet, and extension
|
|
* availability from the generated enabled-extensions set.
|
|
*/
|
|
export function useSettingsNavItems(): { items: SettingsNavItem[]; groups: SettingsNavGroup[] } {
|
|
const { company, isSandbox } = useCompany()
|
|
const { identity } = useAgentSheet()
|
|
const t = useTranslations('settings_nav')
|
|
|
|
const hasCompany = !!company
|
|
const hasBankingExtension = ENABLED_EXTENSION_IDS.has('enable-banking')
|
|
const hasMcpExtension = ENABLED_EXTENSION_IDS.has('mcp-server')
|
|
|
|
// Företagsprofil (TIC-snapshot) lives under Företag; Skatteverket under Skatt;
|
|
// assistentens minne + kunskap under Assistenten; säkerhetsbackup under
|
|
// Importera/Exportera. Team stays hidden (show:false) until enabled.
|
|
const defs: Array<SettingsNavItem & { show: boolean }> = [
|
|
{ id: 'account', href: '/settings/account', label: t('account'), group: 'account', show: true },
|
|
{ id: 'billing', href: '/settings/billing', label: t('billing'), group: 'account', show: true },
|
|
{ id: 'company', href: '/settings/company', label: t('company'), group: 'company', show: hasCompany },
|
|
{ id: 'bookkeeping', href: '/settings/bookkeeping', label: t('bookkeeping'), group: 'accounting', show: hasCompany },
|
|
{ id: 'tax', href: '/settings/tax', label: t('tax'), group: 'accounting', show: hasCompany },
|
|
// Lön settings follow the sidebar: every aktiebolag, plus any company that
|
|
// has registered as an employer (pays_salaries): e.g. an enskild firma
|
|
// with staff. #782
|
|
{ id: 'salary', href: '/settings/salary', label: t('salary'), group: 'accounting', show: hasCompany && (company?.entity_type === 'aktiebolag' || !!company?.pays_salaries) },
|
|
{ id: 'invoicing', href: '/settings/invoicing', label: t('invoicing'), group: 'sales', show: hasCompany },
|
|
{ id: 'templates', href: '/settings/templates', label: t('templates'), group: 'sales', show: hasCompany },
|
|
{ id: 'banking', href: '/settings/banking', label: t('banking'), group: 'tools', show: hasCompany && !isSandbox && hasBankingExtension },
|
|
{ id: 'assistant', href: '/settings/assistant', label: t('assistant'), group: 'tools', show: hasCompany && identity.isVerified },
|
|
{ id: 'api', href: '/settings/api', label: t('api'), group: 'tools', show: hasCompany && hasMcpExtension },
|
|
]
|
|
|
|
const items: SettingsNavItem[] = defs
|
|
.filter((d) => d.show)
|
|
.map(({ show: _show, ...item }) => item)
|
|
|
|
const groupLabels: Record<SettingsGroupKey, string> = {
|
|
account: t('group_account'),
|
|
company: t('group_company'),
|
|
accounting: t('group_accounting'),
|
|
sales: t('group_sales'),
|
|
tools: t('group_tools'),
|
|
}
|
|
|
|
const groups: SettingsNavGroup[] = GROUP_ORDER.map((key) => ({
|
|
key,
|
|
label: groupLabels[key],
|
|
items: items.filter((i) => i.group === key),
|
|
})).filter((g) => g.items.length > 0)
|
|
|
|
return { items, groups }
|
|
}
|