| Age | Commit message (Collapse) | Author |
|
- Implement ServePackageImportedBy handler and GetImportedBy and GetImportedByCount datasource methods.
Change-Id: I8c4cc65fbff7172eaf48e5426e4f3f41c82bd38e
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/754865
Reviewed-by: Jonathan Amsterdam <jba@google.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Ethan Lee <ethanalee@google.com>
|
|
- Introduce a new ServePackageSymbols handler that utilizes the new
db.GetSymbols method.
- Consolidated module resolution logic shared by ServePackages into
resolveModulePath.
- Updated Datasource with GetSymbols, which provides a more efficient
way to retrieve all symbols for a package at a given version and build
context. This differs from the existing getPackageSymbols, since that
will return symbols for the latest release/compatible version.
- Refactored packageSymbolQueryJoin into a generic helper by removing
hardcoded version filters. This allows GetSymbols to query for any
specific version, while getPackageSymbols was updated to explicitly
include release-only filters to preserve its existing behavior.
- GetSymbols still utilizes the same underlying packageSymbolQueryJoin,
but it will also attempt to match the most relevant BuildContext for
the query. It will also provide a mapping of parent and child symbols.
Change-Id: Ib18d2511d24ac6bc5b75c7b3809c4ce126245036
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/754864
Auto-Submit: Ethan Lee <ethanalee@google.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
- Implement the ServeModulePackages handler and introduce
GetModulePackages to return the list of packages for a given module
version.
Change-Id: I20c1618e2fdbf0126cb913665a1d7457b8951177
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/754863
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
Auto-Submit: Ethan Lee <ethanalee@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
- Implement module versions endpoint using ds.GetVersionsForPath
- Introduce paginate helper to generalize paginaton logic
- Update fakedatasource to correctly use V1Path to return all versions
Change-Id: Icc028bf8ca9c13978bb6eba84afe9736ccd6bcee
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/754862
Reviewed-by: Jonathan Amsterdam <jba@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Ethan Lee <ethanalee@google.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
|
|
This CL contains several UI improvements when running pkgsite in local
mode via cmd/pkgsite:
- hide irrelevant or inaccurate content
- fix a bug using -gopath_mode (trim the `go env GOPATH` output)
- fix panics navigating to vuln pages
- link the GO button to the root page, rather than go.dev
- add quick-links to the homepage to browse local modules
Also:
- add -dev and -static flags, to facilitate development
- replace TestBuildGetters with more cases in TestServer
- fix some redundancy in setting the <title> element for pages;
consolidate on using the basePage.HTMLTitle value
Updates golang/go#40371
Change-Id: I459a0d0fd39897dd4f902dd023aec653a3fb12cd
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/475755
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
Run-TryBot: Robert Findley <rfindley@google.com>
|
|
This change adds a new goPackagesModuleGetter, which uses
x/tools/go/packages to query package information for modules requested
by the pkgsite command.
Additionally, add new extension interfaces that allow getters to add
support for search and content invalidation. The go/packages getter uses
these extensions to implement search (via a simple fuzzy-matching
algorithm copied from x/tools), and invalidation via statting package
files.
Along the way, refactor slightly for testing ergonomics.
Updates golang/go#40371
Updates golang/go#50229
Fixes golang/go#54479
Change-Id: Iea91a4d6327707733cbbc4f74a9d93052f33e295
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/474295
TryBot-Result: kokoro <noreply+kokoro@google.com>
Run-TryBot: Robert Findley <rfindley@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
|
|
If GetUnit is passed a build context, it will read documentation
source only for that build context. It still reads all the build
context values, just only one serialized blob of file ASTs.
This sped up getUnitWithAllFields by about 2x on net/http, using
detail-stats against the staging DB from a local frontend.
Change-Id: Ib8e988940b7f8f8ef2c270129c4338bb3d56b9eb
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/312269
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
|
|
If the frontend has called GetUnitMeta(unknown, latest), then
the GetLatestInfo doesn't have to repeat that call.
Change-Id: I61d1b733ded5a588468b8d18db1b66bd027c0fec
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/306594
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
Add a single method to DataSource for getting all information
about latest versions.
Remove the old method.
Use the new method throughout.
Change-Id: Ie1c4894261021e91c9b889d00d98a2c7754017fd
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/279792
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
Currently, the pkgsite allows the user to click a link to be taken
to the latest major version for that module. When a user is in a sub
package, clicking this link transports them to the top of the
directory, rather than the latest major version for that package.
Changes the behaviour of the link by directing to the latest
major version of the currently viewed package. If the package
does not exist in the latest major version, it falls back to
the root of the module.
The direct proxy datasource implementation does not attempt to
perform this resolution and maintains the current behaviour.
Fixes golang/go#42292
Change-Id: I6c17978034f87ceddb0edeae3894ce2cd4913fd2
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/274413
Trust: Julie Qiu <julie@golang.org>
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
Show a module's README links on all units within the module.
For golang/go#42968
Change-Id: I8701a04a16e1dd766b1b23ce12d52d01190840fd
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/275276
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
For golang/go#39629
Change-Id: I8cdcd08818df1e80ac1d77646d892533f257ce3d
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/258288
Trust: Julie Qiu <julie@golang.org>
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
This change creates a function in the DataSource interface that
retrieves the nested modules as ModuleInfos given a path to a directory.
Note that proxydatasource is included since it is required to fill the
datasource interface but only returns an empty slice since we are not
implementing the functionality there.
This change also adds a scanModuleInfo helper function for scanning
ModuleInfo from the db.
Updates golang/go#38596
Change-Id: I02f25aad2640a0828e58d622d1c8661e6a303b84
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/253742
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
GetLicenses is now replaced with GetUnit, using the WithLicenses
fieldset.
GetLicenses is renamed to LegacyGetLicenses, since it is only used in
the legacy code.
For golang/go#39629
Change-Id: I142822ff1f30f1da77d4fda6455d46e5b52a2f35
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/252319
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
This change creates the HTML and CSS changes necessary to show the
latest major version banner.
before (desktop): https://i.imgur.com/YaVw9c8.png
after (desktop): https://i.imgur.com/3XVQPlT.png
before (mobile): https://i.imgur.com/gj54j1o.png
after (mobile): https://i.imgur.com/Xl68EjN.png
Fixes golang/go#37765
Change-Id: I60d6818385c986f9d1ab4dcc8ff663da0e9fe3a3
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251817
Reviewed-by: Julie Qiu <julie@golang.org>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
|
|
PathInfo is renamed to UnitMeta, and GetPathInfo is renamed to
GetUnitMeta.
For golang/go#39629
Change-Id: Ia91cf7d2752988653efcc0e46a4d6b8b63017ef3
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251978
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
GetDirectoryMeta is no longer used and is deleted.
For golang/go#39629
Change-Id: I688013a66317efb418cda7298f59f3225ca239a7
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251977
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
For golang/go#39629
Change-Id: I210b767582bdc7e5bb3adae72a3e4d9a83be5ab5
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251268
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
DataSource methods to be deprecated are grouped together.
For golang/go#39629
Change-Id: I454a1d979418782bc9f03d577f8b5bcc6428c2c4
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251267
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
For golang/go#39629
Change-Id: Iedefbc03b24c6ce2c1b99e3148c0f757a726b687
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251162
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
For golang/go#39629
Change-Id: I5275442a15f066b4c48325880771703a2a77f249
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251164
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
For golang/go#39629
Change-Id: I7abe21764a58b6295926c6cd56ab327824570e77
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251163
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
GetDirectory is now used to fetch data from package_imports in
internal/frontend, instead of GetImports. GetImports is renamed to
LegacyGetImports.
For golang/go#39629
Change-Id: I9655a8e450131e022259835f947ff66dae5d7d9c
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251339
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
The argment "field" is renamed to "fields", when referring to an
internal.FieldSet.
For golang/go#39629
Change-Id: I432d262a6170d7dbee036956edb48aa4607f8af0
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251341
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
This change creates a function in the DataSource interface that
retrieves the latest major version of a module path. This includes a
function in the proxydatasource package.
Updates golang/go#37765
Change-Id: I29505c1d7df51dd9bef8228e73bbedeea9771347
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250797
Reviewed-by: Julie Qiu <julie@golang.org>
Run-TryBot: Julie Qiu <julie@golang.org>
|
|
GetDirectory now supports the WithDocumentation, WithReadme, and
AllFields fieldsets.
At the moment, only one fieldset can be specified at a time. Additional
support will be updated in future CLs.
For golang/go#39629
Change-Id: I42b15d2d06a0b36aa0298a79ea1c06303b682926
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250946
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
For golang/go#39629
Change-Id: I7458dd6555d5de7f2cb01261498064364bb63356
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250947
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
The signature of GetDirectory is changed to:
GetDirectory(ctx context.Context, fullPath, modulePath, version string,
pathID int, fields []internal.FieldSet) (*internal.Directory, error)
The allows the caller to pass in a pathID, if known, and specify
specific fields that they want to fetch, for optimization.
No functional changes to GetDirectory are made in this CL, just an
update to the DatsSource interface, and how it is used in
internal/frontend.
For golang/go#39629
Change-Id: Ic44225a13bf2cf04a62af652dbb830c4d256cb37
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250799
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
GetDirectoryMeta is added to internal/proxydatasource and the DataSource
interface.
For golang/go#39629
Change-Id: I67b7056f3939b319df99bfb083e6d853157e7809
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250542
Reviewed-by: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
|
|
For golang/go#39629
Change-Id: I17171838c864c1708eab527caa2288c57b6b4def
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250506
Reviewed-by: Jamal Carvalho <jamal@golang.org>
|
|
This change introduces a new method on the DataSource interface,
GetLicenses, which returns the licenses that apply for a given path,
module path, and resolved version combination.
A license in the current or any parent directory of the specified path
applies to it.
Fixes golang/go#40027
Change-Id: If91429ac12880ac3b6254bf0acd7d3ac983c93e7
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/241718
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
The New suffix is removed from PackageNew and DirectoryNew, since the
legacy methods are already prefixed with "Legacy".
For golang/go#39629
Change-Id: I8be32310c6b451035021663000aac6ca845ee6e0
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/241900
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
The GetPsuedoVersions* and GetTaggedVersions* functions read from the
packages table, and will be replaced by a single GetVersionsForPath
function.
These functions are now prefixed with "Legacy" to indicate that they
will be deprecated.
For golang/go#39629
Change-Id: I7f89f9890f135b5ddb363a51a9706e48d02594b5
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/241899
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
GetModuleInfo returns the ModuleInfo for given module_path and version.
This will replace LegacyGetModuleInfo.
Fixes golang/go#40032
Change-Id: I9f24de94b93f9f21a29e4f8ca624efc0909d46d5
GitHub-Last-Rev: 5ea5b16579b0aa73f45f7a71db64663b0f74cb76
GitHub-Pull-Request: golang/pkgsite#2
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/241038
Reviewed-by: Julie Qiu <julie@golang.org>
|
|
This is the first CL to clean up various TODOs that are outdated, and
replace internal issue links with GitHub issue links.
Updates golang/go#39621
Change-Id: If270e8b2e8198c007cb4aa71ad8486182f4f3380
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238319
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
The following functions now return *internal.ModuleInfo instead of
*internal.LegacyModuleInfo:
* GetPseudoVersionsForModule
* GetPseudoVersionsForPackageSeries
* GetTaggedVersionsForModule
* GetTaggedVersionsForPackageSeries
Change-Id: Ie62d58998e910f0ee8dca7690729032b1ccac5c3
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238099
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: Ifa83e3bb400902a9c163c0911a361425fceca1c6
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238098
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: I64cada2235d68a1b3fb75b6ba73a921beb7f81f4
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238027
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: Ia9123b6551d950a7ecfdf808e3da5c788512d60a
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238026
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: I3502ba02c79a39c71c5e1a775244f0beebd3cc94
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238025
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: I9964baab0f47aa8dae8a9e09eb8a30741eaaaa43
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238024
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: I05339642aaa7f22ff0074eef99681953574b5bdd
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238023
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
At the moment there are several methods not implemented by the
proxydatasource. These are removed from internal.DataSource.
Fixes b/150138536
Change-Id: Id1eef4b2497bd46c8e3cecf8a083ef81bfbe2f47
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/768688
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
ModuleInfo is renamed to LegacyModuleInfo, as a step towards
deprecating LegacyReadmeFilePath and LegacyReadmeContents.
In a follow up CL, we will add ModuleInfo as an embedded struct
to LegacyModuleInfo.
Change-Id: Ie452420448eec1d13edaf62e548df0b9e2cbbe4b
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/766479
CI-Result: Cloud Build <devtools-proctor-result-processor@system.gserviceaccount.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
The following structs have been renamed with a Legacy prefix:
* internal.Directory
* internal.Package
* internal.VersionedPackage
* sample.Package
The following fields on internal.ModuleInfo have also been changed:
* ReadmeFilePath
* ReadmeFileContents
This is done to help us distinguish between legacy and method
structs/methods while migrating code to the new data model.
Change-Id: Ibedf71d4db6323ef5aa05d73a0240537ea6073d3
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/765160
CI-Result: Cloud Build <devtools-proctor-result-processor@system.gserviceaccount.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Methods on internal.Datasource are rearranged, to group together the
legacy methods that will be deprecated. This is done to help keep track
of which methods need to be removed once the frontend migration is
complete.
Change-Id: I176743d89b8eef8f2334301f6eefafe469c30ab4
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/763560
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Use the results of postgres.GetDirectoryNew to build
package pages.
Change-Id: Iff812416f4fb879baa16536639a80810fc1248b2
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/747780
Reviewed-by: Julie Qiu <julieqiu@google.com>
|
|
Updates b/149933479.
Change-Id: I8c34d62ce216aaa30c79b171981bd190de2eb936
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/747786
Reviewed-by: Julie Qiu <julieqiu@google.com>
|
|
Change-Id: If30847b901fd8044b3b2cdb1f131a2eb260e5436
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/745162
CI-Result: Cloud Build <devtools-proctor-result-processor@system.gserviceaccount.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
|
|
Change-Id: Ia57812b9d5dd7f1b763acd946be9c8cba14d3acc
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/731800
CI-Result: Cloud Build <devtools-proctor-result-processor@system.gserviceaccount.com>
Reviewed-by: Julie Qiu <julieqiu@google.com>
|